Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9492 )
Change subject: python3 support for some build tools ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/9492/3/build-support/push_to_asf.py File build-support/push_to_asf.py: http://gerrit.cloudera.org:8080/#/c/9492/3/build-support/push_to_asf.py@86 PS3, Line 86: print("the contributor guide (git remote add gerrit %s)." % GERRIT_URL, file=sys.stderr) > Mind breaking this out into a separate patch? I know it's tiny but I think This is Python3 compatibility as is: 'rint >>' construct does not work in Python3. Or the idea is to separate build-related and aux stuff? -- To view, visit http://gerrit.cloudera.org:8080/9492 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I547d870a54c7d396d2706d63f65844df01ff7c57 Gerrit-Change-Number: 9492 Gerrit-PatchSet: 3 Gerrit-Owner: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Grant Henke <granthe...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Mon, 05 Mar 2018 19:36:35 +0000 Gerrit-HasComments: Yes