Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23799 )
Change subject: IMPALA-14990: Close Beeswax port by default ...................................................................... IMPALA-14990: Close Beeswax port by default This patch close beeswax port by setting the beeswax_port=0 by default. BE, EE, and CustomCluster tests are modified to only test beeswax if ENABLE_BEESWAX env var is 'true' (or beeswax_port > 0). Skip checking impala-server.num-open-beeswax-sessions in metric_verifier.py if ENABLE_BEESWAX is not 'true'. test_beeswax.py is now moved from regular EE tests to CustomCluster tests because running it will require passing --enable_beeswax flag as start_args. Drop Beeswax option from perf-AB scripts (tests/performance/), and fix unused pluggin_runner arg bug in HiveHS2QueryConfig. Also fixed various flake8 issues in modified python files. Testing: - Pass core tests. - Pass core tests with ENABLE_BEESWAX=true. Change-Id: I4f88b0c9799c8d2244fdc1432bd4ace09d6b87dd Reviewed-on: http://gerrit.cloudera.org:8080/23799 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Joe McDonnell <[email protected]> --- M be/src/observe/otel-test.cc M be/src/rpc/thrift-server-test.cc M be/src/service/impala-server.cc M be/src/service/internal-server-test.cc M be/src/service/session-expiry-test.cc M bin/run-workload.py M bin/start-impala-cluster.py M fe/src/test/java/org/apache/impala/customcluster/LdapImpalaShellTest.java M fe/src/test/java/org/apache/impala/customcluster/LdapKerberosImpalaShellTest.java M tests/common/cluster_config.py M tests/common/environ.py M tests/common/impala_cluster.py M tests/common/impala_test_suite.py M tests/common/skip.py M tests/common/test_dimensions.py R tests/custom_cluster/test_beeswax.py M tests/custom_cluster/test_coordinators.py M tests/custom_cluster/test_kill_query.py M tests/custom_cluster/test_kudu.py M tests/custom_cluster/test_otel_trace.py M tests/custom_cluster/test_query_log.py M tests/custom_cluster/test_restart_services.py M tests/custom_cluster/test_server_tls.py M tests/custom_cluster/test_session_expiration.py M tests/custom_cluster/test_set_and_unset.py M tests/custom_cluster/test_shell_interactive_reconnect.py M tests/performance/query_exec_functions.py M tests/performance/query_executor.py M tests/performance/workload_runner.py M tests/shell/test_shell_commandline_jwt_auth.py M tests/shell/test_shell_interactive.py M tests/shell/util.py M tests/verifiers/metric_verifier.py 33 files changed, 242 insertions(+), 290 deletions(-) Approvals: Impala Public Jenkins: Verified Joe McDonnell: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/23799 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I4f88b0c9799c8d2244fdc1432bd4ace09d6b87dd Gerrit-Change-Number: 23799 Gerrit-PatchSet: 15 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]>
