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

    https://github.com/apache/storm/pull/2526#discussion_r163073567
  
    --- Diff: docs/index.md ---
    @@ -80,7 +80,7 @@ But small change will not affect the user experience. We 
will notify the user wh
     * [Distributed RPC](Distributed-RPC.html)
     * [Transactional topologies](Transactional-topologies.html)
     * [Hooks](Hooks.html)
    -* [Metrics](Metrics.html)
    +* [Metrics](metrics_v2.html)
    --- End diff --
    
    Maybe keeping deprecated metrics doc would help, though we would want to be 
clear that they're deprecated. 
    There's explanation regarding built-in metrics in previous metrics doc, but 
it may look odd to get built-in metrics into metrics V2 doc, because only small 
set of metrics are available in metrics V2 while we don't migrate them. Users 
still need metrics V1 for them.


---

Reply via email to