Tim Armstrong has posted comments on this change.

Change subject: IMPALA-3225: Add script to push from gerrit to ASF
......................................................................


Patch Set 1:

(1 comment)

The bigger problem than branch naming is that the ASF repo's commit hashes 
won't match, since it has all the binaries stripped out. If create a gerrit 
'master' branch that mirrors the ASF 'master' branch, I think that solves both 
problems once we start developing on the ASF codebase directly.

In the short term I'd need to modify my script to propagate changes to both 
gerrit/master and asf/master (or potentially propagate changes to gerrit/master 
then use the script to propagate them to asf/master).

http://gerrit.cloudera.org:8080/#/c/3468/1/bin/push_to_asf.py
File bin/push_to_asf.py:

Line 219:   print Colors.GREEN + ("%d commit(s) need to be pushed from Gerrit 
to ASF:" % len(commits)) + Colors.RESET
Long lines here and elsewhere.


-- 
To view, visit http://gerrit.cloudera.org:8080/3468
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I15de939acc4b08a8f01511fffcb6ab1743d09c01
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Matthew Jacobs <[email protected]>
Gerrit-Reviewer: Matthew Jacobs <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: Yes

Reply via email to