Surya Hebbar has posted comments on this change. ( http://gerrit.cloudera.org:8080/22560 )
Change subject: IMPALA-13805: Fix TSAN build failures for the runtime-profile-test ...................................................................... Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/22560/3/be/src/util/runtime-profile.cc File be/src/util/runtime-profile.cc: http://gerrit.cloudera.org:8080/#/c/22560/3/be/src/util/runtime-profile.cc@2975 PS3, Line 2975: for (AggEventSequence& agg_event_sequen > Move this declaration inside the loop of L2976. Had thought I corrected this, got confused with L1414 and missed. Thanks. Done. -- To view, visit http://gerrit.cloudera.org:8080/22560 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ibdea503d032d456746e9cb4f19d0057197ddb117 Gerrit-Change-Number: 22560 Gerrit-PatchSet: 5 Gerrit-Owner: Surya Hebbar <sheb...@cloudera.com> Gerrit-Reviewer: Fang-Yu Rao <fangyu....@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Surya Hebbar <sheb...@cloudera.com> Gerrit-Comment-Date: Thu, 27 Feb 2025 15:44:23 +0000 Gerrit-HasComments: Yes
