Fix stale configuration documentation. (apache#1257)

efeg · Jul 8, 2020 · c85eb6e · c85eb6e
1 parent e6a1347
commit c85eb6e
Show file tree

Hide file tree

Showing 3 changed files with 165 additions and 165 deletions.
diff --git a/...rc/main/java/com/linkedin/kafka/cruisecontrol/config/constants/AnomalyDetectorConfig.java b/...rc/main/java/com/linkedin/kafka/cruisecontrol/config/constants/AnomalyDetectorConfig.java
@@ -94,8 +94,7 @@ private AnomalyDetectorConfig() {
    */
   public static final String ANOMALY_NOTIFIER_CLASS_CONFIG = "anomaly.notifier.class";
   public static final String DEFAULT_ANOMALY_NOTIFIER_CLASS = NoopNotifier.class.getName();
-  public static final String ANOMALY_NOTIFIER_CLASS_DOC = "The notifier class to trigger an alert when an "
-      + "anomaly is violated.";
+  public static final String ANOMALY_NOTIFIER_CLASS_DOC = "The notifier class to trigger an alert when an anomaly is violated.";
 
   /**
    * <code>anomaly.detection.goals</code>

diff --git a/...rol/src/main/java/com/linkedin/kafka/cruisecontrol/servlet/parameters/ParameterUtils.java b/...rol/src/main/java/com/linkedin/kafka/cruisecontrol/servlet/parameters/ParameterUtils.java
@@ -303,14 +303,16 @@ private static boolean getBooleanExcludeGiven(HttpServletRequest request, String
   }
 
   /**
-   * Default: {@code false} -- i.e. a recently demoted broker may receive leadership from the other brokers.
+   * Default: {@code false} -- i.e. recently demoted brokers may receive leadership from the other brokers as long as
+   * {@link ExecutorConfig#DEMOTION_HISTORY_RETENTION_TIME_MS_CONFIG}.
    */
   static boolean excludeRecentlyDemotedBrokers(HttpServletRequest request) {
     return excludeBrokers(request, EXCLUDE_RECENTLY_DEMOTED_BROKERS_PARAM, false);
   }
 
   /**
-   * Default: {@code true} -- i.e. a recently removed broker may receive replicas from the other brokers.
+   * Default: {@code true} -- i.e. recently removed brokers may not receive replicas from the other brokers as long as
+   * {@link ExecutorConfig#REMOVAL_HISTORY_RETENTION_TIME_MS_CONFIG}.
    */
   static boolean excludeRecentlyRemovedBrokers(HttpServletRequest request) {
     return excludeBrokers(request, EXCLUDE_RECENTLY_REMOVED_BROKERS_PARAM, true);