Re: [PATCH v3 0/7] odb: introduce object storage extension

[Patrick Steinhardt <ps@pks.im>]

On Mon, Oct 26, 2026 at 04:35:27AM -0700, Karthik Nayak wrote:
> Patrick Steinhardt <ps@pks.im> writes:
> 
> > Hi,
> >
> > this patch series introduces the object storage extension. The extension
> > is of course similar in spirit and design to the ref storage extension:
> > it can be configured via "extensions.objectStorage", has the same syntax
> > with "<schema>[://<payload>]", and we have the same mechanisms to
> > configure it ("--object-storage-format=", GIT_OBJECT_STORAGE_FORMAT,
> > GIT_DEFAULT_OBJECT_STORAGE_FORMAT, "init.defaultObjetcStorageFormat").
> >
> > For now, we only support the "files" object storage format, so not a lot
> > of stuff really changes. That being said, its optional payload allows
> > you to configure a different location for the ".git/objects" directory.
> > So if you specify "extensions.objectStorage=files://objs", objects would
> > be stored in ".git/objs" instead.
> >
> > The series is built on top of c51408889c (The 6th batch, 2026-10-16)
> > with the following dependencies merged into it:
> >
> >   - ps/odb-files-alternates at 0ba59c8453 (odb/source: drop
> >     `read_alternates` callback, 2026-10-08) merged into it.
> >
> >   - ps/setup-enforce-repo-passed-to-create-repository-has-no-state
> >     (phew, what a mouthful) at 0ba59c8453 (odb/source: drop
> >     `read_alternates` callback, 2026-10-08).
> >
> > Note that these two series conflict with one another. The conflict can
> > easily be resolved by just deleting the whole conflicting sections.
> >
> 
> I went through the patches since the range-diff was on the larger side.
> Apart from some nits, the series looks good. I think we can ignore my
> nits too :)

Thanks! I'll wait a bit and send out a new version tomorrow morning.

Patrick