uce commented on a change in pull request #6733: [FLINK-10291] Generate
JobGraph with fixed/configurable JobID in StandaloneJobClusterEntrypoint
URL: https://github.com/apache/flink/pull/6733#discussion_r219690410
##########
File path:
flink-container/src/main/java/org/apache/flink/container/entrypoint/StandaloneJobClusterEntryPoint.java
##########
@@ -67,6 +68,8 @@
@Nonnull
private final SavepointRestoreSettings savepointRestoreSettings;
+ private final JobID jobID = new JobID(Long.MIN_VALUE, Long.MIN_VALUE);
Review comment:
The resulting `JobID` for this is: `80000000000000008000000000000000`.
What do you think about using `new JobID(0, 0)` to give an all zero `JobID`
instead? Since this is a fixed ID anyways, I think going with the simplest ID
is preferable and an all zero ID might be easier to use when manually working
with the REST API of a job instead of remembering to insert an `8` (which also
looks very similar to a `0`).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
