godfreyhe commented on a change in pull request #14103:
URL: https://github.com/apache/flink/pull/14103#discussion_r530734736



##########
File path: 
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/environment/StreamExecutionEnvironment.java
##########
@@ -2330,4 +2330,8 @@ public void registerCachedFile(String filePath, String 
name, boolean executable)
                }
                return (T) resolvedTypeInfo;
        }
+
+       private String getJobName() {
+               return configuration.getString(PipelineOptions.NAME, 
DEFAULT_JOB_NAME);

Review comment:
       Thanks all for the suggestions. @twalthr, I have checked that the 
pipeline options are also used in `ExecutionEnvironment`. I also don't notice 
that before, I will update it. The default name of `ExecutionEnvironment` is 
`"Flink Java Job at " + Calendar.getInstance().getTime()`, so the `NAME` option 
has no default value.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to