Jason Fehr has posted comments on this change. ( http://gerrit.cloudera.org:8080/24123 )
Change subject: IMPALA-14796: Show effective runtime filter targets in profile ...................................................................... Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/24123/2/be/src/runtime/coordinator.cc File be/src/runtime/coordinator.cc: http://gerrit.cloudera.org:8080/#/c/24123/2/be/src/runtime/coordinator.cc@821 PS2, Line 821: find_effective_filters(child); Any concerns with stack overflows because of this recursion? http://gerrit.cloudera.org:8080/#/c/24123/2/be/src/runtime/coordinator.cc@1481 PS2, Line 1481: // Also updates the final filter table. : lock_guard<shared_mutex> lock(filter_routing_table_->lock); // Exclusive lock. : ComputeEffectiveFilterTargets(); : if (filter_routing_table_->num_filters() > 0) { : query_profile_->AddInfoString("Final filter table", FilterDebugString()); : } Nit: Surround this block with braces to ensure the compiler does not apply an optimization that moves the locking earlier. -- To view, visit http://gerrit.cloudera.org:8080/24123 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iccf4b87ac4579a70273f3306ec7b58850f06b17c Gerrit-Change-Number: 24123 Gerrit-PatchSet: 2 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Surya Hebbar <[email protected]> Gerrit-Comment-Date: Wed, 25 Mar 2026 23:10:29 +0000 Gerrit-HasComments: Yes
