Re: Patch review process

2015-01-26 Thread Andrew Wang
Let's move this over to common-dev@, general@ is normally used for project announcements rather than discussion topics. I'd like to summarize a few things mentioned on the private@ thread, related to streamlining the code submission process. - Gerrit was brought up again, as it has in the past,

Re: Patch review process

2015-01-26 Thread Ted Yu
In some cases, contributor responded to review comments and attached patches addressing the comments. Later on, there was simply no response to the latest patch - even with follow-on ping. I wish this aspect can be improved. Cheers On Sun, Jan 25, 2015 at 6:03 PM, Tsz Wo (Nicholas), Sze

Re: Patch review process

2015-01-26 Thread Steve Loughran
On 26 January 2015 at 02:03, Tsz Wo (Nicholas), Sze s29752-hadoopgene...@yahoo.com.invalid wrote: I would like to (re)start a discussion regrading to our patch review process. A similar discussion has been happened in a the hadoop private mailing list, which is inappropriate. owning up to