Re: [PR] [FLINK-33236][config] Deprecate the unused high-availability.zookeeper.pth.running-registry option [flink]

["XComp (via GitHub)" <gi...@apache.org>]

XComp commented on code in PR #23506:
URL: https://github.com/apache/flink/pull/23506#discussion_r1354598520


##########
flink-core/src/main/java/org/apache/flink/configuration/HighAvailabilityOptions.java:
##########
@@ -167,12 +167,6 @@ public class HighAvailabilityOptions {
                     .withDescription(
                             "Defines the number of connection retries before the client gives up.");
 
-    @Documentation.Section(Documentation.Sections.EXPERT_ZOOKEEPER_HIGH_AVAILABILITY)

Review Comment:
   Yeah, I see your point. The worst thing that could happen as far as I can think is that the field is used in some downstream project. I guess that would be a reason why we would follow the proper deprecation process. The only documentation I found about that is what's stated in the kind of old [Best Practices and Lessons Learned](https://cwiki.apache.org/confluence/display/FLINK/Best+Practices+and+Lessons+Learned)



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

To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org