sanpwc commented on code in PR #5249:
URL: https://github.com/apache/ignite-3/pull/5249#discussion_r1967555549
##########
modules/partition-replicator/src/integrationTest/java/org/apache/ignite/internal/partition/replicator/ItReplicaLifecycleTest.java:
##########
@@ -579,7 +579,7 @@ public void testTableEstimatedSize() throws Exception {
});
// Read the key from another transaction to trigger write intent
resolution, and so incrementing the estimated size.
- // TODO https://issues.apache.org/jira/browse/IGNITE-24360
Perhaps, it should be reworked some way
+ // TODO https://issues.apache.org/jira/browse/IGNITE-24361
Perhaps, it should be reworked some way
Review Comment:
Given code
```
node.transactions().runInTransaction(tx -> {
keyValueView1.getAll(tx, kv1.keySet());
keyValueView2.getAll(tx, kv2.keySet());
});
```
is no longer needed. WI will be converted because of proper tx finish,
explicit WI Resolution through gets not required.
--
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: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
