Jason Fehr has posted comments on this change. ( http://gerrit.cloudera.org:8080/23293 )
Change subject: IMPALA-13237: [Patch 6] - Match Behavior of Older Impala Versions ...................................................................... Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/23293/5//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/23293/5//COMMIT_MSG@7 PS5, Line 7: IMPALA-13237: > IMPALA-13237: [patch 6] Done http://gerrit.cloudera.org:8080/#/c/23293/5/tests/custom_cluster/test_otel_trace.py File tests/custom_cluster/test_otel_trace.py: http://gerrit.cloudera.org:8080/#/c/23293/5/tests/custom_cluster/test_otel_trace.py@42 PS5, Line 42: BEESWAX > Can we drop BEESWAX? We are deprecating it, and it will cut number of tests I'm fine with doing that. My motivation with including beeswax was that queries executed via beeswax take different paths through the code which could lead to unexpected behavior if not tested. However, on second thought, Impala could only support queries as OTel traces when clients connect via HS2. -- To view, visit http://gerrit.cloudera.org:8080/23293 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If57aaef2da6d6904c66d0150f50ea9ac1c3ebc8c Gerrit-Change-Number: 23293 Gerrit-PatchSet: 5 Gerrit-Owner: Jason Fehr <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Pranav Lodha <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Mon, 18 Aug 2025 18:18:32 +0000 Gerrit-HasComments: Yes
