ptupitsyn commented on code in PR #6059:
URL: https://github.com/apache/ignite-3/pull/6059#discussion_r2157144992
##########
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:
This class is reused on the client side where `observableTimestamp` is not
needed. We should probably use a different class in that case. I'll keep this
in mind.
--
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
