Mryange opened a new pull request, #28814:
URL: https://github.com/apache/doris/pull/28814

   ## Proposed changes
   
   pipeline logic
   
   ``` C++
   _sink->channel_all_can_write();
   
   for (auto channel : _channels) {
           if (!channel->can_write()) {
                   return false;
           }
   }
   
   bool can_write() {
           if (!is_local()) {
               return true;
           }
   
           // if local recvr queue mem over the exchange node mem limit, we 
must ensure each queue
           // has one block to do merge sort in exchange node to prevent the 
logic dead lock
           return !_local_recvr || _local_recvr->is_closed() || 
!_local_recvr->exceeds_limit(0) ||
                  _local_recvr->sender_queue_empty(_parent->sender_id());
       }
   ```
   
   <!--Describe your changes.-->
   
   ## Further comments
   
   If this is a relatively large or complex change, kick off the discussion at 
[d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you 
chose the solution you did and what alternatives you considered, etc...
   
   


-- 
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

Reply via email to