Github user zd-project commented on the issue:

    https://github.com/apache/storm/pull/2789
  
    I can't see changes in your link, so I looked into your latest change in 
branch `non-static-metrics`. Please correct me if I interpret your code wrong.
    
    Your implementation didn't actually flush the metrics for ScheduledReporter 
on exit. You want to add `reporter.report()` in ConsolePreparableReporter or 
CsvPreparableReporter and test it out. See my commit [STORM-3173: Addressing 
nits](https://github.com/apache/storm/pull/2789/commits/e5fb9d80f4af8fee2b3cff915c5401a1e0e0cde2#diff-65a7d8c48dacfbb072a99f50df9775c6R40)


---

Reply via email to