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


---

Reply via email to