eaglewatcherwb commented on a change in pull request #7412: 
[FLINK-10866][Runtime] 1. Explicitly enable qs server and proxy. 2. QS
URL: https://github.com/apache/flink/pull/7412#discussion_r245540045
 
 

 ##########
 File path: 
flink-core/src/main/java/org/apache/flink/configuration/QueryableStateOptions.java
 ##########
 @@ -106,6 +106,12 @@
                        .defaultValue(0)
                        .withDescription("Number of query Threads for queryable 
state server. Uses the number of slots if set to 0.");
 
+       /** Enable Queryable state server and proxy. */
+       public static final ConfigOption<Boolean> 
ENABLE_QUERYABLE_STATE_PROXY_SERVER =
+               key("query.enable")
 
 Review comment:
   ENABLE_QUERYABLE_STATE_PROXY_SERVER is used to configure both QS proxy and 
server, so I think query.enable is more meaningful.
   Make description more readable is a good idea, I will change it.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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