From: Karthik Nayak <karthik.188@gmail.com>
Date: Thu, 30 Oct 2025 12:24:02 GMT
Subject: Re: [PATCH 00/13] Carve out loose object source
Message-ID: <CAOLa=ZTr-9==Xs1G39EbjKaT3PDOVTY5uDXiJYmzEwqpZLdY6Q@mail.gmail.com>
In-Reply-To: <20251024-b4-pks-odb-loose-backend-v1-0-1a4202273c38@pks.im>

Patrick Steinhardt <ps@pks.im> writes:

> Hi,
>
> this patch series carves out loose object sources. The idea is to store
> all data that relates to loose objects in a single structure, similar to
> our `struct packfile_store`.
>
> The series is structured as follows:
>
>   - Patches 1 to 4 perform some cleanups in the vicinity of object
>     databases.
>
>   - Patches 5 to 8 create a new `struct odb_loose_source` and move all
>     state that is specific to loose objects into it.
>
>   - Patches 9 to 13 then adjust functions to work on top of that new
>     structure.
>
> The motivation here is to make handling of loose objects completely
> self-contained as a step towards pluggable object databases.
>
> Thanks!
>
> Patrick
>

The series looks good in its current form, I only had one comment. Will
leave it to if it needs to be re-rolled :)

Karthik