Jason Fehr has posted comments on this change. ( http://gerrit.cloudera.org:8080/22443 )
Change subject: IMPALA-13726 Add admission control slots to /queries page in webui ...................................................................... Patch Set 3: Code-Review+1 (9 comments) http://gerrit.cloudera.org:8080/#/c/22443/2/be/src/service/query-state-record-test.cc File be/src/service/query-state-record-test.cc: http://gerrit.cloudera.org:8080/#/c/22443/2/be/src/service/query-state-record-test.cc@170 PS2, Line 170: 4 > Done Done http://gerrit.cloudera.org:8080/#/c/22443/2/testdata/workloads/functional-query/queries/QueryTest/workload-mgmt-impala_query_live-v1.2.0.test File testdata/workloads/functional-query/queries/QueryTest/workload-mgmt-impala_query_live-v1.2.0.test: http://gerrit.cloudera.org:8080/#/c/22443/2/testdata/workloads/functional-query/queries/QueryTest/workload-mgmt-impala_query_live-v1.2.0.test@58 PS2, Line 58: 'orderby_columns','string','' > This test result list is missing coordinator_slots and executor_slots. Done http://gerrit.cloudera.org:8080/#/c/22443/2/testdata/workloads/functional-query/queries/QueryTest/workload-mgmt-impala_query_log-v1.2.0.test File testdata/workloads/functional-query/queries/QueryTest/workload-mgmt-impala_query_log-v1.2.0.test: http://gerrit.cloudera.org:8080/#/c/22443/2/testdata/workloads/functional-query/queries/QueryTest/workload-mgmt-impala_query_log-v1.2.0.test@58 PS2, Line 58: 'orderby_columns','string',NULL > This test result list is missing coordinator_slots and executor_slots. Done http://gerrit.cloudera.org:8080/#/c/22443/2/tests/custom_cluster/test_workload_mgmt_init.py File tests/custom_cluster/test_workload_mgmt_init.py: http://gerrit.cloudera.org:8080/#/c/22443/2/tests/custom_cluster/test_workload_mgmt_init.py@176 PS2, Line 176: 1.1.0 > Done Done http://gerrit.cloudera.org:8080/#/c/22443/2/tests/custom_cluster/test_workload_mgmt_init.py@188 PS2, Line 188: "1.2.0", vector, m > yes, thanks Done http://gerrit.cloudera.org:8080/#/c/22443/2/tests/custom_cluster/test_workload_mgmt_init.py@200 PS2, Line 200: # Verify the initial table create on version 1.0.0 succeeded. > Done Done http://gerrit.cloudera.org:8080/#/c/22443/2/tests/custom_cluster/test_workload_mgmt_init.py@220 PS2, Line 220: def test_upgrade_1_1_0_to_1_2_0(self, vector): > Done Done http://gerrit.cloudera.org:8080/#/c/22443/2/tests/custom_cluster/test_workload_mgmt_init.py@257 PS2, Line 257: self.assert_catalogd_all_tables(r"Workload management table '{}' is at version " > Done Done http://gerrit.cloudera.org:8080/#/c/22443/2/tests/util/workload_management.py File tests/util/workload_management.py: http://gerrit.cloudera.org:8080/#/c/22443/2/tests/util/workload_management.py@618 PS2, Line 618: # The first host has the coordinator admission slots. > Tidied up scope of these variables Done -- To view, visit http://gerrit.cloudera.org:8080/22443 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I057493b7767902a417dfeb75cdaeffd452d66789 Gerrit-Change-Number: 22443 Gerrit-PatchSet: 3 Gerrit-Owner: Andrew Sherman <[email protected]> Gerrit-Reviewer: Andrew Sherman <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Comment-Date: Tue, 04 Feb 2025 21:47:14 +0000 Gerrit-HasComments: Yes
