yiguolei commented on code in PR #14371:
URL: https://github.com/apache/doris/pull/14371#discussion_r1029021685
##########
be/src/vec/runtime/shared_hash_table_controller.cpp:
##########
@@ -22,23 +22,24 @@
namespace doris {
namespace vectorized {
-bool SharedHashTableController::should_build_hash_table(RuntimeState* state,
int my_node_id) {
+bool SharedHashTableController::should_build_hash_table(const TUniqueId&
fragment_instance_id,
+ int my_node_id) {
std::lock_guard<std::mutex> lock(_mutex);
auto it = _builder_fragment_ids.find(my_node_id);
if (it == _builder_fragment_ids.cend()) {
Review Comment:
If the builder instance's hash join node closed, should remove its id from
the builder fragments and signal all waiters.
In wait for hash table method, should check if the builder fragments still
exists in case of the builder fragments exist during errors.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
