New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add nearcaching hook to map-transaction-log-record #16581
Conversation
e837b63
to
e450c75
Compare
run-lab-run |
@@ -175,18 +175,41 @@ public void commit() throws TransactionException, IllegalStateException { | |||
} | |||
} | |||
|
|||
public void commitAsync(ExecutionCallback callback) { | |||
public void commitAsync(final ExecutionCallback callback) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we add these too?
https://github.com/hazelcast/hazelcast/pull/16579/files#diff-0c104d8009708b16b9092edfd2438fa1R173-R176
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
didn't get this?
} | ||
}; | ||
|
||
void beforeRemoteCall(K key, Data keyData, V value, Data valueData); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How come you don't use @Nullable
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It wasn't common in 3.12 (java 6) using those, no real reason.
public MapTransactionLogRecord() { | ||
} | ||
|
||
public MapTransactionLogRecord(String name, Data key, int partitionId, Operation op, long version, String ownerUuid) { | ||
public MapTransactionLogRecord(String name, Data key, int partitionId, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can the parameter be@Nullable
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same
run-lab-run |
e450c75
to
d31358d
Compare
closes #16577
Backport of: #16579