Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/24317 )
Change subject: IMPALA-14843: (addendum) fixed broken test for Calcite planner ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/24317/2/tests/query_test/test_observability.py File tests/query_test/test_observability.py: http://gerrit.cloudera.org:8080/#/c/24317/2/tests/query_test/test_observability.py@a216 PS2, Line 216: > Unfortuantely, Calcite does not have the alias there and I haven't looked i The alias is fine. Let's keep this comment unchanged, i.e. don't change 6 to 3 and don't change 4 to 1. -- To view, visit http://gerrit.cloudera.org:8080/24317 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I930a9a2f52dd727388c9e9eb70b6452f8343d4c8 Gerrit-Change-Number: 24317 Gerrit-PatchSet: 3 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]> Gerrit-Comment-Date: Mon, 18 May 2026 12:06:34 +0000 Gerrit-HasComments: Yes
