From: Patrick Steinhardt <ps@pks•im>
To: Junio C Hamano <gitster@pobox•com>
Cc: git@vger•kernel.org, Justin Tobler <jltobler@gmail•com>,
Taylor Blau <me@ttaylorr•com>
Subject: Re: What's cooking in git.git (May 2025, #07; Fri, 23)
Date: Tue, 27 May 2025 10:15:11 +0200 [thread overview]
Message-ID: <aDV0jwaQ2DlcM0lZ@pks.im> (raw)
In-Reply-To: <xmqqtt5au523.fsf@gitster.g>
On Fri, May 23, 2025 at 07:16:04PM -0700, Junio C Hamano wrote:
> * ps/midx-negative-packfile-cache (2025-05-20) 2 commits
> - midx: stop repeatedly looking up nonexistent packfiles
> - packfile: explain ordering of how we look up auxiliary pack files
>
> When a stale .midx file refers to .pack files that no longer exist,
> we ended up checking for these non-existent files repeatedly, which
> has been optimized by memoizing the non-existence.
>
> Will merge to 'next'?
> source: <20250520-pks-pack-avoid-stats-on-missing-v2-0-333c5217fb05@pks•im>
I wanted to send one more iteration of this where I hide the ugliness of
`(void *)(intptr_t)-1` behind a macro, as suggested. But I saw that
Taylor has built on top of these patches, so I don't want to make his
life harder. Cc'd him.
> * ps/object-store (2025-05-14) 18 commits
> - odb: rename `read_object_with_reference()`
> - odb: rename `pretend_object_file()`
> - odb: rename `has_object()`
> - odb: rename `repo_read_object_file()`
> - odb: rename `oid_object_info()`
> - odb: trivial refactorings to get rid of `the_repository`
> - odb: get rid of `the_repository` when handling submodule alternates
> - odb: get rid of `the_repository` when handling the primary alternate
> - odb: get rid of `the_repository` in `for_each()` functions
> - odb: get rid of `the_repository` when handling alternates
> - odb: get rid of `the_repository` in `odb_mkstemp()`
> - odb: get rid of `the_repository` in `assert_oid_type()`
> - odb: get rid of `the_repository` in `find_odb()`
> - odb: introduce parent pointers
> - object-store: rename files to "odb.{c,h}"
> - object-store: rename `object_directory` to `odb_alternate`
> - object-store: rename `raw_object_store` to `object_database`
> - Merge branch 'ps/object-store-cleanup' into ps/object-store
>
> Code clean-up around object access API.
>
> Comments?
> source: <20250514-pks-object-store-wo-the-repository-v3-0-47df1d4ead22@pks•im>
I think the only outstanding discussion is whether to name things
`odb_alternate` or `odb_source` [1]. In case others agree that
`odb_source` is a better name I'm happy to revise, but if not I'd rather
keep it as-is.
Other than that I think the patch series is in a good shape.
Thanks!
Patrick
[1]: <tjsbotrnrffykmi3letktpb3bly4nqw4wxzyrszgbln7pznem4@3kwiq4zvaebw>
next prev parent reply other threads:[~2025-05-27 8:15 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-24 2:16 What's cooking in git.git (May 2025, #07; Fri, 23) Junio C Hamano
2025-05-27 8:15 ` Patrick Steinhardt [this message]
2025-05-27 16:50 ` Junio C Hamano
2025-05-27 19:45 ` Justin Tobler
2025-05-30 9:47 ` Patrick Steinhardt
2025-05-30 16:28 ` Junio C Hamano
2025-06-02 6:42 ` Patrick Steinhardt
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aDV0jwaQ2DlcM0lZ@pks.im \
--to=ps@pks$(echo .)im \
--cc=git@vger$(echo .)kernel.org \
--cc=gitster@pobox$(echo .)com \
--cc=jltobler@gmail$(echo .)com \
--cc=me@ttaylorr$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox