kangpinghuang commented on a change in pull request #2055: add delete state of 
row block v2 for performance
URL: https://github.com/apache/incubator-doris/pull/2055#discussion_r344590226
 
 

 ##########
 File path: be/src/olap/generic_iterators.cpp
 ##########
 @@ -262,6 +266,9 @@ Status MergeIterator::next_batch(RowBlockV2* block) {
         // copy current row to block
         copy_row(&dst_row, ctx->current_row(), block->pool());
 
+        if (ctx->is_partial_delete()) {
+            block->set_delete_state(DEL_PARTIAL_SATISFIED);
 
 Review comment:
   Here the block is not filtered yet, should should be set delete status.
   Delete can be calculated here, but need to refactor condition to realize it. 
which will be done in another pr.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to