Andrew Sherman 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 2: (7 comments) Thanks for the quick review 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: 7 > Nit: I suggest making this value '8' to assert the get_admission_slots func 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.0.0 > Nit: should be "1.1.0" Done http://gerrit.cloudera.org:8080/#/c/22443/2/tests/custom_cluster/test_workload_mgmt_init.py@188 PS2, Line 188: self.LATEST_SCHEMA > I would prefer to pass in the string "1.2.0" here only because modifying th yes, thanks 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. > Nit: unneessary outdent 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): > Please add an additional test for upgrading from 1.0.0 to 1.2.0. Done http://gerrit.cloudera.org:8080/#/c/22443/2/tests/custom_cluster/test_workload_mgmt_init.py@257 PS2, Line 257: # Run a query and ensure it does not populate version 1.2.0 fields. > Nit: should say something like "does not populate fields other than version 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: expected_executor_slots = "0" > Please consider moving the calculation of expected_coordinator_slots and ex Tidied up scope of these variables -- 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: 2 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 01:03:52 +0000 Gerrit-HasComments: Yes
