StephanEwen commented on issue #10982: [FLINK-15683][docs] Restructure 
Configuration page
URL: https://github.com/apache/flink/pull/10982#issuecomment-580943075
 
 
   Pushed a fix addressing the latest comments.
   
   > If the ssl.* options aren't supposed to be used anymore they should 
properly deprecated, likely with deprecated keys on the new options, as should 
the web option, as should `local.number-resourcemanager` since (apparently) it 
isn't even used anymore.
   
     - I think the SSL options are slightly more complex to resolve than just 
with deprecated keys. That's why I went with only removing them from the config 
section for now.
     - The `local.number-resourcemanager` is indeed not used any more, added a 
`@Deprecated` instead.
     - The `web.address` option should have been deprecated before (probably 
overlooked back then). As far as I can tell they are also unused (only 
referenced as fallback keys).

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


With regards,
Apache Git Services

Reply via email to