git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Elijah Newren <newren@gmail.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: Christian Couder <christian.couder@gmail.com>,
	Johannes Schindelin <Johannes.Schindelin@gmx.de>,
	John Cai <johncai86@gmail.com>, git <git@vger.kernel.org>
Subject: Re: What's cooking in git.git (Aug 2023, #01; Wed, 2)
Date: Mon, 7 Aug 2023 09:56:12 -0700	[thread overview]
Message-ID: <CABPp-BE5h6C1iuGDz4HdJzb2bDLidEnQ0_U4fv93JrYXA2_cbw@mail.gmail.com> (raw)
In-Reply-To: <xmqqv8dqhlkz.fsf@gitster.g>

On Mon, Aug 7, 2023 at 9:19 AM Junio C Hamano <gitster@pobox.com> wrote:
>
[...]
> >> However in a recent article
> >> (https://github.blog/2023-07-27-scaling-merge-ort-across-github/)
> >> GitHub says that they are already using git-replay (though not sure
> >> it's this exact version or another one), and GitLab plans to use it
> >> too.
>
> So there are plenty folks who are capable of reviewing but they are
> not interested in giving it to the general public by withholding
> their reviews ;-)?

I can see how one could jump to that conclusion, but I don't think
it's warranted.  I have a little information that might shed some
light:

-----

At the beginning of July, well before the above link came out,
Johannes sent me an email pointing me at
https://github.blog/changelog/2023-06-28-rebase-commits-now-created-using-the-merge-ort-strategy/.
In the email, he also noted my earlier stated concerns on the list
(about releasing `replay` early possibly painting us into a corner
preventing some desired goals for the tool from being realized), and
tried to ensure we could still work towards having a `replay` command
like the one I envisioned while also asking for my thoughts on pushing
for the current portion of work to be published and included in git.
My sense was he was pushing for the work to be released, but just
being careful about my concerns first.

Unfortunately, I was on vacation that week, and sadly have otherwise
been buried in coming up to speed on a new team and haven't had time
to even respond to git-related stuff.  I didn't respond to him until
late last week.

I pointed out that the 'EXPERIMENTAL' banner addresses most of my
concerns so it should be fine to move forward, but I suspect my delay
has resulted in him now being buried in other matters, and in the
mean-time the article Christian highlighted was produced by others.

-----

Anyway, hope that helps.  I did review V2 and have been meaning to
comment on V3 (and respond to Toon's comments), though not sure my
"review" counts as such since I'm the original author of most of the
patches...

(And sorry for being missing in action.  I've flagged a few other
emails that I'm hoping I can follow up on, but my time is currently
quite limited...)

  parent reply	other threads:[~2023-08-07 16:56 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-02 18:10 What's cooking in git.git (Aug 2023, #01; Wed, 2) Junio C Hamano
2023-08-02 19:09 ` jt/path-filter-fix (was: Re: What's cooking in git.git (Aug 2023, #01; Wed, 2)) Taylor Blau
2023-08-02 19:20   ` jt/path-filter-fix Junio C Hamano
2023-08-02 19:28     ` jt/path-filter-fix Taylor Blau
     [not found] ` <CAP8UFD1vuB2kRr890B7GXum9HAMjRep86zy2tE4yE2C3W5QGHA@mail.gmail.com>
2023-08-07 12:41   ` What's cooking in git.git (Aug 2023, #01; Wed, 2) Christian Couder
2023-08-07 16:19     ` Junio C Hamano
2023-08-07 16:22       ` Taylor Blau
2023-08-07 16:56       ` Elijah Newren [this message]
2023-08-17 13:21         ` Johannes Schindelin
2023-08-07 20:59 ` Rubén Justo
2023-08-08 19:22 ` leak in jt/path-filter-fix, was " Jeff King
2023-08-08 19:25   ` Jeff King
2023-08-08 20:22   ` Taylor Blau
2023-08-08 21:19     ` Jeff King
2023-08-11 22:15     ` Jonathan Tan

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=CABPp-BE5h6C1iuGDz4HdJzb2bDLidEnQ0_U4fv93JrYXA2_cbw@mail.gmail.com \
    --to=newren@gmail.com \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=christian.couder@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=johncai86@gmail.com \
    /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).