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:
[email protected]
