Re: [PATCH v5 4/7] update-index: use the bulk-checkin infrastructure
- From
- Neeraj Singh <nksingh85@gmail.com>
- Date
- Sep 24, 2021, 21:49 UTC
- Message-ID
- <CANQDOdfZrn0YK0_HomzqHkqnxmjXc20aa6TPkmwpiapeGpjjyw@mail.gmail.com>
- In-Reply-To
- <485b4a767dfa54729c40b32b7fea033aedc870d1.1632514331.git.gitgitgadget@gmail.com>
On Fri, Sep 24, 2021 at 1:12 PM Neeraj Singh via GitGitGadget <gitgitgadget@gmail.com> wrote:
Show 11 quoted lines
> > From: Neeraj Singh <neerajsi@microsoft.com> > > The update-index functionality is used internally by 'git stash push' to > setup the internal stashed commit. > > This change enables bulk-checkin for update-index infrastructure to > speed up adding new objects to the object database by leveraging the > pack functionality and the new bulk-fsync functionality. This mode > is enabled when passing paths to update-index via the --stdin flag, > as is done by 'git stash'.
This part of the description is now inaccurate. All modes of update-index are now enlightened to use bulk_checkin. I'll just remove the sentence that scopes the change to --stdin on reroll.
Show 11 quoted lines
> > There is some risk with this change, since under batch fsync, the object > files will not be available until the update-index is entirely complete. > This usage is unlikely, since any tool invoking update-index and > expecting to see objects would have to synchronize with the update-index > process after passing it a file path. > > Signed-off-by: Neeraj Singh <neerajsi@microsoft.com> > --- > builtin/update-index.c | 6 ++++++ > 1 file changed, 6 insertions(+)