Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2686#discussion_r191043657
--- Diff:
storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java ---
@@ -371,6 +372,8 @@
private static final List<String> EMPTY_STRING_LIST =
Collections.unmodifiableList(Collections.emptyList());
private static final Set<String> EMPTY_STRING_SET =
Collections.unmodifiableSet(Collections.emptySet());
private static final Pattern TOPOLOGY_NAME_REGEX =
Pattern.compile("^[^/.:\\\\]+$");
+ private static final RotatingMap<String, Long> topologyCleanupDetected
= new RotatingMap<String, Long>(2);
--- End diff --
Nit: Could use `<>` on the right hand side
---