Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/20079 )
Change subject: [build] Change shebangs to python3 ...................................................................... Patch Set 1: > Patch Set 1: > > > > Patch Set 1: > > > > > > > (1 comment) > > > > > > It would be great to keep compatibility -- all existing Kudu > > Python scripts work with Python2 as well if they work with Python3. > > We have spent time and effort on that, and I'm not sure we want to > > abandon that effort in the nearest future since we still have bunch > > of systems that on Python2. > > > > > > Yes, Python community decided to abandon Python2 because they > > don't want to spent resources on maintaining compatibility, but > > that's not in our interests as of now. > > > > While I do not agree that we should keep Python2 compatibility > > alive, this change is not user facing. The changes only affect > > thirdparty and build-support python files, so users who run Kudu on > > environments with Python2 would not be affected. I believe the > > number of end-users who compile their own Kudu is rather small, and > > upgrading a build environment to also run Python3 (e.g. with pyenv) > > should not be as impactful as upgrading on edge/client nodes. > > Adding a symlink or running update-alternatives isn't a big deal either. > > The affected people are Kudu developers, and I one of those. If you don't do > much development in Kudu on platforms which are still python2-based, that's > fine, but there are other people around you who will be affected. It wouldn't be a big deal, but on my fresh install of MacOS Ventura that didn't work. I symlinked python3 to python, and when I tried to run python, it tried to install CLI developer tools, and after it installed them, it still wouldn't run. -- To view, visit http://gerrit.cloudera.org:8080/20079 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I824daa0cf151e89f783db98d46fb23ce9e267093 Gerrit-Change-Number: 20079 Gerrit-PatchSet: 1 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Wang Xixu <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Thu, 22 Jun 2023 06:53:05 +0000 Gerrit-HasComments: No
