dant3 commented on code in PR #7095:
URL: https://github.com/apache/ignite-3/pull/7095#discussion_r2580232661
##########
modules/table/src/test/java/org/apache/ignite/internal/table/distributed/StorageUpdateHandlerTest.java:
##########
@@ -222,4 +226,44 @@ void testUpdateAllBatchedTryLockFailed() {
assertEquals(row3, result3.binaryRow());
}
+ /**
+ * Tests that {@link StorageUpdateHandler#handleUpdateAll} respects {@code
shouldRelease()} from the storage engine.
+ *
+ * <p>This test verifies that the implementation checks {@code
shouldRelease()} during batch processing
+ * to allow the storage engine to perform critical operations like
checkpoints.
Review Comment:
Yeah, I somehow missed committing the code for `shouldRelease` change piece.
Fixed.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]