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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
