impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Apple <>
Subject Re: Switch gerrit merge strategy to "rebase always"?
Date Mon, 11 Dec 2017 19:55:05 GMT
Is there a merge strategy that just doesn't allow commit chains longer than


"When cherry picking a change, Gerrit automatically appends onto the end of
the commit message a short summary of the change’s approvals, and a URL
link back to the change on the web. The committer header is also set to the
submitter, while the author header retains the original patch set author."

I love that short message. It's useful to be able to easily see the code
review comments and reviewer names.

On Mon, Dec 11, 2017 at 11:43 AM, Tim Armstrong <>

> We recently had a bad merge that was allowed by the cherry-pick merge
> strategy merging a simple without its ancestor (since they didn't change
> any nearby lines):
> It looks like the "rebase always" merge strategy avoids this by always
> trying to rebase and merge the whole chain of commits. Does anyone have any
> objections or thoughts about switching to this strategy?

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message