samza-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chris Riccomini <>
Subject Re: commit style
Date Thu, 15 Aug 2013 23:41:01 GMT
Hey Jay,

Good point.

I suspect RB will prefer the squashed style. I think JIRA will prefer this
as well (SAMZA-14.0.patch), right? What do other projects do?


On 8/15/13 4:36 PM, "Jay Kreps" <> wrote:

>We have been using sort of different git styles. I think Chris has been
>checking in rapidly. I do lots of quick commits but then rebase it into
>giant patch for review.
>Do we have a preference? I am fine either way. For code review the quick
>commit style is actually nicer because the git format-patch command will
>then actually show the original patch plus the changes made in response to
>feedback each as different commits. The problem with this is that your
>history gets pretty hard to understand. The big patch way gives a very
>clean version of history but makes code review a bit harder.
>I am cool with either or both.
>Not sure how git format-patch interacts with review board...

View raw message