Sahil Takiar has posted comments on this change. ( http://gerrit.cloudera.org:8080/16408 )
Change subject: IMPALA-10129 Data race in MemTracker::GetTopNQueriesAndUpdatePoolStats ...................................................................... Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/16408/4/be/src/runtime/mem-tracker.h File be/src/runtime/mem-tracker.h: http://gerrit.cloudera.org:8080/#/c/16408/4/be/src/runtime/mem-tracker.h@97 PS4, Line 97: bool is_query_mem_tracker = false, const TUniqueId& query_id = TUniqueId()); is it necessary for these parameters to all have default values? creating a default query_id of just TUniqueId() seems dangerous -- To view, visit http://gerrit.cloudera.org:8080/16408 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9c4ffe8064d3e099a525cc48c218ef73112fb67b Gerrit-Change-Number: 16408 Gerrit-PatchSet: 4 Gerrit-Owner: Qifan Chen <[email protected]> Gerrit-Reviewer: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Sahil Takiar <[email protected]> Gerrit-Comment-Date: Thu, 03 Sep 2020 15:08:38 +0000 Gerrit-HasComments: Yes
