samza-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chris Riccomini <>
Subject Re: commit style
Date Sat, 17 Aug 2013 03:03:13 GMT
Hey Guys,

Reading over:

I definitely prefer a squashed .patch on JIRA, and a squashed RB, as well.
The un-squashed v2 patch on SAMZA-14 is a bit hard to follow.

Whether we squash them on commit, I'm undecided on.


On 8/15/13 4:41 PM, "Chris Riccomini" <> wrote:

>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
>>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