thomasrebele commented on code in PR #6100: URL: https://github.com/apache/hive/pull/6100#discussion_r2376519722
########## data/scripts/cat.py: ########## @@ -16,6 +16,7 @@ # specific language governing permissions and limitations # under the License. # +from __future__ import print_function Review Comment: I included this on purpose to make the script executable with both, python 2 and python 3. This is to avoid issues if the CI is still using python2. As python2 is not available for my OS anymore, I've used https://onecompiler.com/python to check whether both support `from __future__ import print_function`. I just noticed there is another issue: python3 does not support `has_key` anymore. I've updated the PR to fix it for cat.py and input20_script.py (and checked whether they run on my laptop). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
