[GitHub] storm pull request #2735: STORM-3121: Fix flaky metrics tests in storm-core

[revans2]

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

    https://github.com/apache/storm/pull/2735#discussion_r197838595
  
    --- Diff: storm-core/pom.xml ---
    @@ -184,6 +184,10 @@
                 <artifactId>mockito-core</artifactId>
                 <scope>test</scope>
             </dependency>
    +        <dependency>
    +            <groupId>org.awaitility</groupId>
    +            <artifactId>awaitility</artifactId>
    +        </dependency>
    --- End diff --
    
    nit: like in the other pull request, it would be nice to have this marked 
as test here too, just so it is obvious.


---