Steve Carlin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24322 )
Change subject: IMPALA-15001: OpenTelemetry Query Tracing not working with fallback planner ...................................................................... IMPALA-15001: OpenTelemetry Query Tracing not working with fallback planner Some tests within test_otel_trace.py are failing when the USE_CALCITE_PLANNER environment variable is set to true (i.e. test_select_values). This fix avoids setting the tracing when parsing has failed, since the Calcite planner may have issues with parsing that are resolved by the original planner. A "finally" block is added outside of all retries to ensure that the backend gets a message for all requests. Change-Id: Ia776e7bc3245b950d332855c59207d5f21e26bff Reviewed-on: http://gerrit.cloudera.org:8080/24322 Reviewed-by: Jason Fehr <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/service/Frontend.java 1 file changed, 9 insertions(+), 10 deletions(-) Approvals: Jason Fehr: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/24322 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ia776e7bc3245b950d332855c59207d5f21e26bff Gerrit-Change-Number: 24322 Gerrit-PatchSet: 3 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]> Gerrit-Reviewer: Surya Hebbar <[email protected]>
