git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Neeraj Singh <nksingh85@gmail.com>
To: Elijah Newren <newren@gmail.com>
Cc: "Junio C Hamano" <gitster@pobox.com>,
	"Ævar Arnfjörð Bjarmason" <avarab@gmail.com>,
	"Git Mailing List" <git@vger.kernel.org>,
	"Jeff King" <peff@peff.net>,
	"Jonathan Nieder" <jrnieder@gmail.com>,
	"Neeraj Singh" <neerajsi@microsoft.com>
Subject: Re: What's cooking in git.git (Sep 2021, #08; Mon, 27)
Date: Fri, 8 Oct 2021 15:30:22 -0700	[thread overview]
Message-ID: <CANQDOdeSjKsGUYacGwx-fnigqiocbyDOfz2O8im6-7txDCuZzQ@mail.gmail.com> (raw)
In-Reply-To: <CABPp-BE=7+XifO3mAzNB90Y0vatvhwk7wwM4ptBrg79BhiFqSg@mail.gmail.com>

Thanks Junio.  I'd be happy to have my change go in first.  I'm hoping
that we'll be able to get a lot of mileage with batch mode in Git For
Windows early on.

Thanks,
Neeraj

On Thu, Oct 7, 2021 at 11:52 PM Elijah Newren <newren@gmail.com> wrote:
>
> On Thu, Oct 7, 2021 at 3:01 PM Junio C Hamano <gitster@pobox.com> wrote:
> >
> > Neeraj Singh <nksingh85@gmail.com> writes:
> >
> > > Elijah,
> > >
> > > Here's a branch of your changes based on the amalgamated tmp-objdir code:
> > > https://github.com/neerajsi-msft/git/commits/neerajsi/remerge-diff
> > >
> > > This commit adapts your code to use the amalgamated API:
> > > https://github.com/neerajsi-msft/git/commit/725328fe1d8be8326d2ddef78e164ca21450b100
> >
> > It seems that the discussion petered out at this point.
> >
> > Right now I have a version of ns/remerge-diff before this adjustment
> > in 'seen', and Neeraj's latest version is kept out of 'seen' as they
> > do not play well together without an adjustment like that.
> >
> > What's the good way forward?  I do not deeply care which one goes
> > first, but I have a feeling that the need by remerge-diff that wants
> > to discard temporary objects would involve more work to make it safe
> > than the need by batched fsync where newly created objects will not
> > be discarded but merely moved to the primary store before the end of
> > the operation, so from that point of view, it seems simpler and
> > safer to queue ns/batched-fsync topic first (especially given that
> > it is a no-op until the end-user opts into the experiment), and have
> > a remerge-diff that uses the infrastructure from Neeraj's topic.
> >
> > What's your take on the rebase Neeraj made, Elijah (at the URL
> > above)?
>
> I meant to dig further, but nearly all my git time in the last week
> and a half was attempting to keep up with other patch reviews.  My git
> time is fast disappearing in the near term, and it's not clear how
> much, if any, time I'll have to work on patches (or even continued
> reviewing) before, say, mid-November.  I most likely won't be able to
> do any discussion-prep work in advance of the Git Contributor's
> Summit, and might not even be able to attend anymore.
>
> I had looked over Neeraj's patches and they looked reasonable.  I
> thought there might be some tweaks that I could try out, but at this
> point, just take what he has and keep my topic as expecting an update.
> I'll circle back eventually.
>
> Sorry for the delay.

  reply	other threads:[~2021-10-08 22:30 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-28  0:52 What's cooking in git.git (Sep 2021, #08; Mon, 27) Junio C Hamano
2021-09-28  1:57 ` Ævar Arnfjörð Bjarmason
2021-09-28 20:52   ` Junio C Hamano
2021-09-28  6:46 ` Elijah Newren
2021-09-28  7:45   ` Ævar Arnfjörð Bjarmason
2021-09-28 17:25     ` Junio C Hamano
2021-09-28 21:00       ` Neeraj Singh
2021-09-28 23:34         ` Junio C Hamano
2021-09-28 23:53           ` Neeraj Singh
2021-10-07 22:01             ` Junio C Hamano
2021-10-08  6:51               ` Elijah Newren
2021-10-08 22:30                 ` Neeraj Singh [this message]
2021-10-08 23:01                 ` Junio C Hamano
2021-09-28  8:07   ` Ævar Arnfjörð Bjarmason
2021-09-28 17:27     ` Junio C Hamano
2021-09-28 13:31   ` Derrick Stolee
2021-09-28 17:33     ` Junio C Hamano
2021-09-28 20:16       ` Derrick Stolee
2021-09-28 17:16   ` Junio C Hamano
2021-09-29  6:42     ` Elijah Newren
2021-09-28 23:40   ` Jeff King
2021-09-28 23:49     ` Jeff King
2021-09-29 18:43     ` Neeraj Singh
2021-09-30  8:16       ` Jeff King
2021-10-01  7:50         ` Elijah Newren
2021-10-01 17:02           ` Junio C Hamano
2021-10-01 17:39             ` Neeraj Singh
2021-10-01 18:15               ` Elijah Newren
2021-10-01 18:12             ` Elijah Newren
2021-10-01 22:02               ` Junio C Hamano
2021-10-01 23:05                 ` Elijah Newren
2021-10-04 13:45     ` Elijah Newren
2021-09-28  8:22 ` da/difftool (was: Re: What's cooking in git.git (Sep 2021, #08; Mon, 27)) Ævar Arnfjörð Bjarmason
2021-09-28  8:23 ` ns/batched-fsync & en/remerge-diff (was " Ævar Arnfjörð Bjarmason
2021-09-28  8:31 ` sg/test-split-index-fix " Ævar Arnfjörð Bjarmason
2021-09-28  8:35 ` hn/reftable (Re: " Ævar Arnfjörð Bjarmason
2021-09-28 12:18   ` Han-Wen Nienhuys
2021-09-30  5:06     ` Carlo Arenas
2021-09-29  8:12 ` What's cooking in git.git (Sep 2021, #08; Mon, 27) Fabian Stelzer
2021-09-30 21:26   ` Junio C Hamano

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=CANQDOdeSjKsGUYacGwx-fnigqiocbyDOfz2O8im6-7txDCuZzQ@mail.gmail.com \
    --to=nksingh85@gmail.com \
    --cc=avarab@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=jrnieder@gmail.com \
    --cc=neerajsi@microsoft.com \
    --cc=newren@gmail.com \
    --cc=peff@peff.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
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;
as well as URLs for NNTP newsgroup(s).