Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/23343 )
Change subject: IMPALA-14348: Fix TSAN issue after IMPALA-14327 ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/23343/1/be/src/service/impala-hs2-server.cc File be/src/service/impala-hs2-server.cc: http://gerrit.cloudera.org:8080/#/c/23343/1/be/src/service/impala-hs2-server.cc@647 PS1, Line 647: query_handle->returns_result_set()); Can result_metadata_ actually get modified in parallel? It seems like an error if based on timings this can either return true or false http://gerrit.cloudera.org:8080/#/c/23343/1/be/src/service/impala-hs2-server.cc@673 PS1, Line 673: RETURN_IF_ERROR(query_handle->SetResultCache(result_set, cache_num_rows)); SetResultCache() will also take the lock, and AFAIK std::mutex is not recursive -- To view, visit http://gerrit.cloudera.org:8080/23343 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I41fc25cea5b4ef7b4b9daac54b8665fa76ceb1cd Gerrit-Change-Number: 23343 Gerrit-PatchSet: 1 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Pranav Lodha <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Mon, 25 Aug 2025 05:53:44 +0000 Gerrit-HasComments: Yes
