Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20188 )

Change subject: KUDU-3402 Update trace-viewer with newest catapult changes.
......................................................................


Patch Set 3:

(3 comments)

Thank you for the patch!

http://gerrit.cloudera.org:8080/#/c/20188/3//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/20188/3//COMMIT_MSG@25
PS3, Line 25: kudu
nit: Kudu


http://gerrit.cloudera.org:8080/#/c/20188/3//COMMIT_MSG@25
PS3, Line 25: restor
nit: restore


http://gerrit.cloudera.org:8080/#/c/20188/3//COMMIT_MSG@27
PS3, Line 27: 
https://github.com/cloudera/catapult/commit/003d1462c43c196462f2f317f4d89116c7ca7421
I saw updates in this changelist that aren't compatible with Python2.  Is it 
possible to keep it working on both Python2 and Python3?  We have other scripts 
that work for both version of python, and we haven't switched everything to 
Python3.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I931de59a2cad944bfb945b3e1bee3e9c96b5b34b
Gerrit-Change-Number: 20188
Gerrit-PatchSet: 3
Gerrit-Owner: Zoltan Martonka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Comment-Date: Mon, 17 Jul 2023 23:03:02 +0000
Gerrit-HasComments: Yes

Reply via email to