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



##########
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:
       Are pipeline options also used in `ExecutionEnvironment`? The PR doesn't 
update this class but maybe it should? In that case `Flink Streaming Job` would 
be wrong.




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