[GitHub] storm pull request #2546: STORM-2934 - fix startup ClassNotFoundException wh...

[HeartSaVioR]

Github user HeartSaVioR commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2546#discussion_r166201614
  
    --- Diff: storm-server/src/main/java/org/apache/storm/DaemonConfig.java ---
    @@ -1035,7 +1035,10 @@
          * Class implementing MetricStore.
          */
         @NotNull
    -    @isImplementationOfClass(implementsClass = MetricStore.class)
    +    @isString
    +    //@isImplementationOfClass(implementsClass = MetricStore.class)
    --- End diff --
    
    minor: Let's remove this line then. I think below explanation is enough.


---