ascherbakoff commented on code in PR #6059:
URL: https://github.com/apache/ignite-3/pull/6059#discussion_r2158131937
##########
modules/client-common/src/main/java/org/apache/ignite/internal/compute/ComputeJobDataHolder.java:
##########
@@ -27,17 +29,30 @@
public class ComputeJobDataHolder {
private final ComputeJobDataType type;
- private final byte[] data;
+ private final byte @Nullable [] data;
+
+ private final @Nullable Long observableTimestamp;
Review Comment:
Do I understand correctly it's not needed when an object constructed on a
job initiator side because we don't need to propagate OTS from a client to a
server ?
Let's at least add a javadoc comment explaining nullable condition.
--
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
