deadwind4 commented on code in PR #19685:
URL: https://github.com/apache/flink/pull/19685#discussion_r869946982


##########
flink-python/pyflink/datastream/stream_execution_environment.py:
##########
@@ -807,17 +807,30 @@ def register_cached_file(self, file_path: str, name: str, 
executable: bool = Fal
         self._j_stream_execution_environment.registerCachedFile(file_path, 
name, executable)
 
     @staticmethod
-    def get_execution_environment() -> 'StreamExecutionEnvironment':
+    def get_execution_environment(configuration: Configuration = None) \
+            -> 'StreamExecutionEnvironment':
         """
         Creates an execution environment that represents the context in which 
the
         program is currently executed. If the program is invoked standalone, 
this
         method returns a local execution environment.
 
+        When executed from the command line the given configuration is stacked 
on top of the
+        global configuration which comes from the flink-conf.yaml, potentially 
overriding
+        duplicated options.
+
+        :param configuration: The configuration to instantiate the environment 
with.
         :return: The execution environment of the context in which the program 
is executed.
         """
         gateway = get_gateway()
-        j_stream_exection_environment = 
gateway.jvm.org.apache.flink.streaming.api.environment\
-            .StreamExecutionEnvironment.getExecutionEnvironment()
+        JStreamExecutionEnvironment = 
gateway.jvm.org.apache.flink.streaming.api.environment \
+            .StreamExecutionEnvironment
+
+        if configuration:
+            j_stream_exection_environment = 
JStreamExecutionEnvironment.getExecutionEnvironment()

Review Comment:
   I find the only one 
[test_table_environment_api.py](https://github.com/apache/flink/pull/19685/files#diff-51b6ef54d67d0bdb98ffaac4535b82acdce2c0e591164a7018350aeda663ecb3)
 can be updated



-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to