Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/3926#discussion_r117180703
  
    --- Diff: docs/dev/stream/queryable_state.md ---
    @@ -208,10 +208,16 @@ Once used in a job, you can retrieve the job ID and 
then query any key's current
     
     {% highlight java %}
     final Configuration config = new Configuration();
    -config.setString(ConfigConstants.JOB_MANAGER_IPC_ADDRESS_KEY, 
queryAddress);
    -config.setInteger(ConfigConstants.JOB_MANAGER_IPC_PORT_KEY, queryPort);
    +config.setString(JobManagerOptions.ADDRESS, queryAddress);
    +config.setInteger(JobManagerOptions.PORT, queryPort);
     
    -QueryableStateClient client = new QueryableStateClient(config);
    +final HighAvailabilityServices highAvailabilityServices =
    +      HighAvailabilityServicesUtils.createHighAvailabilityServices(
    +           config,
    +           TestingUtils.defaultExecutor(),
    +           
HighAvailabilityServicesUtils.AddressResolution.NO_ADDRESS_RESOLUTION);
    --- End diff --
    
    Better to do `TRY_ADDRESS_RESOLUTION` here so that we fail fast in case the 
JM is not reachable from the node we are currently running on.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to