Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19551 )
Change subject: IMPALA-11952 (part 1): Fix except syntax ...................................................................... IMPALA-11952 (part 1): Fix except syntax Python 3 does not support this old except syntax: except Exception, e: Instead, it needs to be: except Exception as e: This uses impala-futurize to fix all locations of the old syntax. Testing: - The check-python-syntax.sh no longer shows errors for except syntax. Change-Id: I1737281a61fa159c8d91b7d4eea593177c0bd6c9 Reviewed-on: http://gerrit.cloudera.org:8080/19551 Reviewed-by: Joe McDonnell <[email protected]> Reviewed-by: Michael Smith <[email protected]> Tested-by: Michael Smith <[email protected]> --- M bin/bootstrap_toolchain.py M bin/generate_xml_config.py M bin/get_code_size.py M bin/start-impala-cluster.py M docker/monitor.py M infra/deploy/deploy.py M testdata/common/cgroups.py M tests/beeswax/impala_beeswax.py M tests/common/impala_cluster.py M tests/common/impala_service.py M tests/common/impala_test_suite.py M tests/custom_cluster/test_coordinators.py M tests/custom_cluster/test_hdfs_timeout.py M tests/custom_cluster/test_hs2_fault_injection.py M tests/custom_cluster/test_local_catalog.py M tests/custom_cluster/test_mem_reservations.py M tests/custom_cluster/test_query_expiration.py M tests/custom_cluster/test_query_retries.py M tests/custom_cluster/test_restart_services.py M tests/metadata/test_ddl.py M tests/metadata/test_hdfs_permissions.py M tests/performance/query_exec_functions.py M tests/query_test/test_beeswax.py M tests/query_test/test_decimal_queries.py M tests/query_test/test_insert.py M tests/query_test/test_nested_types.py M tests/query_test/test_partitioning.py M tests/query_test/test_query_mem_limit.py M tests/query_test/test_udfs.py M tests/shell/test_shell_commandline.py M tests/statestore/test_statestore.py M tests/unittests/test_result_verifier.py 32 files changed, 75 insertions(+), 75 deletions(-) Approvals: Joe McDonnell: Looks good to me, approved Michael Smith: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/19551 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1737281a61fa159c8d91b7d4eea593177c0bd6c9 Gerrit-Change-Number: 19551 Gerrit-PatchSet: 4 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]>
