tkalkirill commented on code in PR #1673:
URL: https://github.com/apache/ignite-3/pull/1673#discussion_r1109520462
##########
modules/storage-page-memory/src/main/java/org/apache/ignite/internal/storage/pagememory/mv/AbortWriteInvokeClosure.java:
##########
@@ -74,10 +70,7 @@ public void call(@Nullable VersionChain oldRow) throws
IgniteInternalCheckedExce
toRemove = latestVersion;
if (latestVersion.hasNextLink()) {
- // Next can be safely replaced with any value (like 0), because
this field is only used when there
- // is some uncommitted value, but when we add an uncommitted
value, we 'fix' such placeholder value
- // (like 0) by replacing it with a valid value.
- newRow = VersionChain.createCommitted(rowId,
latestVersion.nextLink(), NULL_LINK);
+ newRow = VersionChain.createCommitted(rowId,
latestVersion.nextLink(), latestVersion.nextLink());
Review Comment:
Because it didn't work right before.
--
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]
