Jason Fehr has uploaded this change for review. ( http://gerrit.cloudera.org:8080/23413
Change subject: IMPALA-14373: Take ExecState Lock in OpenTelemetry Trace SpanManager ...................................................................... IMPALA-14373: Take ExecState Lock in OpenTelemetry Trace SpanManager According to the documentation in impala-server.h, the exec_state_lock_ must be held while accessing the ExecState from ClientRequestState objects. The two places in the SpanManager class where the ExecState object is accessed now take the exec_state_lock_ in the ClientRequestState object. Testing accomplished by running custom cluster tests locally and in a build. End-to-end tests were also run with OTEL tracing enabled. Change-Id: Ica9bb637516c8acfbee77f082b67d725e4cec5d8 --- M be/src/observe/span-manager.cc M be/src/observe/span-manager.h M be/src/service/client-request-state.h 3 files changed, 9 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/13/23413/5 -- To view, visit http://gerrit.cloudera.org:8080/23413 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ica9bb637516c8acfbee77f082b67d725e4cec5d8 Gerrit-Change-Number: 23413 Gerrit-PatchSet: 5 Gerrit-Owner: Jason Fehr <[email protected]>
