xintongsong commented on a change in pull request #9760: [FLINK-13982][runtime] 
Implement memory calculation logics
URL: https://github.com/apache/flink/pull/9760#discussion_r333803202
 
 

 ##########
 File path: 
flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/HAQueryableStateFsBackendITCase.java
 ##########
 @@ -96,7 +96,7 @@ private static Configuration getConfig() throws Exception {
 
                Configuration config = new Configuration();
                
config.setBoolean(QueryableStateOptions.ENABLE_QUERYABLE_STATE_PROXY_SERVER, 
true);
-               config.setString(TaskManagerOptions.MANAGED_MEMORY_SIZE, "4m");
+               config.setString(TaskManagerOptions.LEGACY_MANAGED_MEMORY_SIZE, 
"4m");
 
 Review comment:
   Not until we remove the feature option and clean up the legacy codes.
   Currently, FLIP-49 is disabled by default. If we use the new key here, it 
will be ignored by the legacy code path. 

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


With regards,
Apache Git Services

Reply via email to