Ádám Bakai has posted comments on this change. ( http://gerrit.cloudera.org:8080/19187 )
Change subject: [python] Make clang_tidy work with python 3.6.9 ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/19187/1/build-support/clang_tidy_gerrit.py File build-support/clang_tidy_gerrit.py: http://gerrit.cloudera.org:8080/#/c/19187/1/build-support/clang_tidy_gerrit.py@79 PS1, Line 79: decode() > I guess this might fail in Python 2.x if the output contains non-ascii char I have no idea what would happen in that case. I'm not sure how realistic it is, considering all of the support ended in 2020 (source: https://www.python.org/doc/sunset-python-2/). It works fine with UTF-8 characters with python3. -- To view, visit http://gerrit.cloudera.org:8080/19187 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6ce1503422152c6ecd27f382795e11f1df64d67f Gerrit-Change-Number: 19187 Gerrit-PatchSet: 1 Gerrit-Owner: Ádám Bakai <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Ádám Bakai <[email protected]> Gerrit-Comment-Date: Thu, 03 Nov 2022 12:17:32 +0000 Gerrit-HasComments: Yes
