Github user HeartSaVioR commented on the issue:

    https://github.com/apache/storm/pull/2203
  
    @revans2 
    I'm OK to deprecate old metrics API in 2.0 and remove it from 3.0. More 
important aspect from me is the following up patches from @abellina.
    
    As I stated earlier, the patch only migrate small parts of existing metrics 
to new metrics. In master branch we should migrate all the metrics internally 
used in Storm codebase to new metrics (do we want to use deprecated API by 
ourselves?) and then built-in metrics will not be compatible with current 
metrics transfer mechanism (adding to heartbeat). 
    
    I expect the patches from @abellina will address this point. Right?


---

Reply via email to