Correct a deprecated method implementation

Signed-off-by: Tianli Feng <[email protected]>
opensearch-project · Jun 21, 2022 · 6c34ae8 · 6c34ae8
1 parent 8fc134d
commit 6c34ae8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 12 deletions.
diff --git a/...src/main/java/org/opensearch/action/support/clustermanager/ClusterManagerNodeRequest.java b/...src/main/java/org/opensearch/action/support/clustermanager/ClusterManagerNodeRequest.java
@@ -85,13 +85,10 @@ public final TimeValue clusterManagerNodeTimeout() {
         return this.clusterManagerNodeTimeout;
     }
 
-    /**
-     * Preserve the method so that classes implements AcknowledgedRequest don't need to override it for backwards compatibility.}
-     *
-     * @deprecated As of 2.1, because supporting inclusive language, replaced by {@link #clusterManagerNodeTimeout()}
-     */
+    // Preserve the method so that classes implements AcknowledgedRequest don't need to override it for backwards compatibility.
+    /** @deprecated As of 2.1, because supporting inclusive language, replaced by {@link #clusterManagerNodeTimeout()} */
     @Deprecated
     public final TimeValue masterNodeTimeout() {
-        return this.clusterManagerNodeTimeout;
+        return clusterManagerNodeTimeout();
     }
 }
diff --git a/.../java/org/opensearch/action/support/clustermanager/TransportClusterManagerNodeAction.java b/.../java/org/opensearch/action/support/clustermanager/TransportClusterManagerNodeAction.java
@@ -120,12 +120,9 @@ protected TransportClusterManagerNodeAction(
     protected abstract void clusterManagerOperation(Request request, ClusterState state, ActionListener<Response> listener)
         throws Exception;
 
-    /**
-     * Preserve the method so that org.opensearch.action.support.master.info.TransportClusterInfoAction class
-     * can override masterOperation() for backwards compatibility.}
-     *
-     * @deprecated As of 2.1, because supporting inclusive language, replaced by {@link #clusterManagerOperation(ClusterManagerNodeRequest, ClusterState, ActionListener)}
-     */
+    // Preserve the method so that o.o.action.support.master.info.TransportClusterInfoAction class
+    // can override masterOperation() for backwards compatibility.
+    /** @deprecated As of 2.1, because supporting inclusive language, replaced by {@link #clusterManagerOperation(ClusterManagerNodeRequest, ClusterState, ActionListener)} */
     @Deprecated
     protected void masterOperation(Request request, ClusterState state, ActionListener<Response> listener) throws Exception {
         clusterManagerOperation(request, state, listener);