tillrohrmann commented on issue #6872: [FLINK-10436] Add 
ConfigOption#withFallbackKeys
URL: https://github.com/apache/flink/pull/6872#issuecomment-431287759
 
 
   I could imagine a custom HTTP client used to query the REST endpoint which 
relies on `RestOptions#ADDRESS`. Admittedly, the `RestOptions` are marked as 
`@Internal` though.
   
   Yes, at some point I guess we have to break the behaviour.
   
   However, I think we are derailing the discussion a bit and widening the 
scope of this issue. The issue is intended to solve the problem that we log 
warnings when falling back to the `JobManagerOptions#ADDRESS`. The dependency 
of `RestOptions#ADDRESS` on `JobManagerOptions#ADDRESS` is not intended to be 
resolved in the scope of this issue. I think the changes of this PR do not make 
things worse and indeed solve the problem of deprecation warnings. 
Consequently, I would suggest that we address the disentanglement in a separate 
issue. What do you think @zentol?
   
   

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