Robert Haas wrote:
>
> If git had a place to store all the information we care about, that
> would be fine...
>
> There's no "reviewer" header, and there's no concept that a patch
> might have come from the author (or perhaps multiple authors), but
> then have been adjusted by one or more reviewers and then frobnicated
> some more by the committer
> ...
> I don't think that non-linear history is an advantage in any
> situation.
ISTM we could have the best of both those worlds - linear history
and author&reviewer&committer information.
Instead of squashing every patch into a single commit, what if it got
squashed into a perhaps 3 separate commits -- one as submitted, one
as reviewed, and one as re-written by the committer. History stays
linear; and you keep the most relevant parts of the history,
while dropping all the development brainstorming commits.
And ISTM the patch reviewer could be responsible for this squashing
so it's not much more work for the committer.
For example, instead of
commit 351c0b92eca40c1a36934cf83fe75db9dc458cde Author: Robert Haas <robertmhaas@gmail.com> Date: Fri Jul 23
00:43:002010 +0000 Avoid deep recursion when assigning XIDs to multiple levels of subxacts. Andres Freund, with
cleanupand adjustment for older branches by me.
we'd see
Author: Andreas Freund Date: Fri Jul 23 00:43:00 2010 +0000 Avoid deep recursion when assigning XIDs to
multiplelevels of subxacts. Path as originally submitted to commit fest
Author: [Whomever the reviewer was] Date: Fri Jul 23 00:43:00 2010 +0000 Avoid deep recursion when assigning
XIDsto multiple levels of subxacts. Patch marked read for committer by reviewer.
Author: Robert Haas <robertmhaas@gmail.com> Date: Fri Jul 23 00:43:00 2010 +0000 Avoid deep recursion when
assigningXIDs to multiple levels of subxacts. Patch as rewritten by committer.
For a complex enough patch with many authors, the reviewer could choose to
keep an extra author commit or two to credit the extra authors.