dianfu commented on a change in pull request #12370: URL: https://github.com/apache/flink/pull/12370#discussion_r433643407
########## File path: flink-python/src/main/java/org/apache/flink/python/PythonOptions.java ########## @@ -148,4 +149,17 @@ "The priority is as following: 1. the configuration 'python.client.executable' defined in " + "the source code; 2. the environment variable PYFLINK_EXECUTABLE; 3. the configuration " + "'python.client.executable' defined in flink-conf.yaml"); + + /** + * Whether the memory used by the Python framework is managed memory. + */ + public static final ConfigOption<Boolean> USE_MANAGED_MEMORY = ConfigOptions Review comment: I'm not quite sure about this. I think it still has its value to expose it to users, e.g. Python worker uses managed memory by default In 1.10 and it allows Python UDF users of 1.10 to migrate their job to 1.11 by setting this config option as `true` to have the same behavior as 1.10. I guess this is also the similar reason why `state.backend.rocksdb.memory.managed` exists. ---------------------------------------------------------------- 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: us...@infra.apache.org