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

   ## Proposed changes
   
   In the StatefulOperator, there is code like this:
   ```
   Status NestLoopJoinProbeOperator::prepare(doris::RuntimeState* state) {
       // just for speed up, the way is dangerous
       _child_block = _node->get_left_block();
       return StatefulOperator::prepare(state);
   }
   ```
   This will lead to a failure in the check because both _origin_block in the 
exec node of the old pipeline and _child_block in the StatefulOperator will 
simultaneously hold the same column.
   Here's the modification: previously, we would clear _origin_block and then 
pull, but here we are switching to pulling first and then clearing 
_origin_block.
   
   
   
   <!--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