Skip to content

Commit 39d6d90

Browse files
reiabreuclaude
andcommitted
Use forRemoval/since on deprecated NIMBUS_SUPERVISOR_TIMEOUT_SECS
Match Storm's existing deprecation convention (e.g. Config.java) by marking the unused constant @deprecated(forRemoval = true, since = "3.0.1") and stating in the javadoc that it is scheduled for removal. 3.0.1 is the current development version (root pom is 3.0.1-SNAPSHOT). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 4f1b4ec commit 39d6d90

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

storm-server/src/main/java/org/apache/storm/DaemonConfig.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -281,9 +281,10 @@ public class DaemonConfig implements Validated {
281281
* @deprecated Unused. Supervisor liveness is tracked via an ephemeral ZooKeeper node (see
282282
* {@code StormClusterState#supervisorHeartbeat}); when a supervisor dies its ZooKeeper session
283283
* expires and the node disappears, so Nimbus detects the loss directly rather than by timing out
284-
* heartbeats. No code reads this value. Retained only for backward compatibility.
284+
* heartbeats. No code reads this value. It is scheduled for removal; retained for now only for
285+
* backward compatibility.
285286
*/
286-
@Deprecated
287+
@Deprecated(forRemoval = true, since = "3.0.1")
287288
@IsInteger
288289
@IsPositiveNumber
289290
public static final String NIMBUS_SUPERVISOR_TIMEOUT_SECS = "nimbus.supervisor.timeout.secs";

0 commit comments

Comments
 (0)