mymeiyi commented on code in PR #32980:
URL: https://github.com/apache/doris/pull/32980#discussion_r1571711166


##########
regression-test/suites/insert_p0/txn_insert.groovy:
##########
@@ -328,19 +480,44 @@ suite("txn_insert") {
                 where txn_insert_dt4.id = txn_insert_dt2.id;
             """
             sql """
-                delete from txn_insert_dt2 where id = 1 or id = 5;
+                delete from txn_insert_dt2 where id = 1;
             """
             sql """
-                delete from txn_insert_dt5 partition(p_20000102) where id = 1 
or id = 5;
+                delete from txn_insert_dt2 where id = 5;
+            """

Review Comment:
   > If we do insert and delete on the same table, then there is a problem 
that, if the delete is a predicate and implemented via push handler, then the 
delete take effect on the insert result. while if the delete is in `delete 
from` format then the delete does not take effect on the insert result.
   > 
   > It is ambiguous for users.
   
   For `delete from t1` and `insert t1` case, the delete does not work the next 
insert, see case 14.



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