ibessonov commented on code in PR #1976:
URL: https://github.com/apache/ignite-3/pull/1976#discussion_r1180310872
##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/StorageUpdateHandler.java:
##########
@@ -189,6 +198,8 @@ public void handleUpdateAll(
RowId rowId = new RowId(partitionId, entry.getKey());
BinaryRow row = entry.getValue() != null ? new
ByteBufferRow(entry.getValue()) : null;
+ locker.lock(rowId);
Review Comment:
Yes, we should. I though I might do that in separate issue, but forgot to
create it and add TODO. Good catch, thank you!
On the other hand, code for sorting is not particularly big, I can do it
with ease
--
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]
