You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2022/07/13 22:20:19 UTC
[kudu-CR] This commit will fix python 3.x failure in dep extract.py
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18384 )
Change subject: This commit will fix python 3.x failure in dep_extract.py
......................................................................
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/18384/2/build-support/dep_extract.py
File build-support/dep_extract.py:
http://gerrit.cloudera.org:8080/#/c/18384/2/build-support/dep_extract.py@108
PS2, Line 108: match = LDD_RE.match(line.decode('utf-8'))
I didn't look deep, but from what I can see this change breaks isolate with errors like
flex_partitioning-itest.7 failed to process isolate (isolateDir: /kudu/build/isolate): invalid character '.' after array element
You can see that in http://jenkins.kudu.apache.org/job/kudu-gerrit/26002/BUILD_TYPE=DEBUG/consoleFull
--
To view, visit http://gerrit.cloudera.org:8080/18384
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ied217653a90a6c4260e889cdfbafa683938acf1e
Gerrit-Change-Number: 18384
Gerrit-PatchSet: 2
Gerrit-Owner: harinreddy <hn...@us.ibm.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <an...@g.ucla.edu>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 13 Jul 2022 22:20:19 +0000
Gerrit-HasComments: Yes