ascherbakoff commented on code in PR #6059:
URL: https://github.com/apache/ignite-3/pull/6059#discussion_r2157003712
##########
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:
It's not clear to me why observable ts can be nullable.
Looks like it must be always provided.
--
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
