[jira] [Commented] (HELIX-785) Report helix latency instead of user latency during top state handoff
[ https://issues.apache.org/jira/browse/HELIX-785?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16673669#comment-16673669 ] Hudson commented on HELIX-785: -- FAILURE: Integrated in Jenkins build helix #1569 (See [https://builds.apache.org/job/helix/1569/]) [HELIX-785] Record helix latency instead of user latency in top state (hrzhang: rev ccf263c9110fa5af54f1c1cabd8b5a2af64d473e) * (edit) helix-core/src/main/java/org/apache/helix/monitoring/mbeans/ClusterStatusMonitor.java * (edit) helix-core/src/test/java/org/apache/helix/monitoring/mbeans/TestTopStateHandoffMetrics.java * (edit) helix-core/src/test/resources/TestTopStateHandoffMetrics.json * (edit) helix-core/src/main/java/org/apache/helix/controller/stages/TopStateHandoffReportStage.java * (edit) helix-core/src/main/java/org/apache/helix/monitoring/mbeans/ResourceMonitor.java > Report helix latency instead of user latency during top state handoff > - > > Key: HELIX-785 > URL: https://issues.apache.org/jira/browse/HELIX-785 > Project: Apache Helix > Issue Type: Task >Reporter: Harry Zhang >Assignee: Harry Zhang >Priority: Major > > Currently we are reporting top state handoff user latency, but we should > report Helix latency instead. user should have their way of monitoring their > own state transitions. > AC: > 1. Implement reporting Helix latency for top state handoff and test it -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (HELIX-785) Report helix latency instead of user latency during top state handoff
[ https://issues.apache.org/jira/browse/HELIX-785?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16673659#comment-16673659 ] ASF GitHub Bot commented on HELIX-785: -- Github user asfgit closed the pull request at: https://github.com/apache/helix/pull/292 > Report helix latency instead of user latency during top state handoff > - > > Key: HELIX-785 > URL: https://issues.apache.org/jira/browse/HELIX-785 > Project: Apache Helix > Issue Type: Task >Reporter: Harry Zhang >Assignee: Harry Zhang >Priority: Major > > Currently we are reporting top state handoff user latency, but we should > report Helix latency instead. user should have their way of monitoring their > own state transitions. > AC: > 1. Implement reporting Helix latency for top state handoff and test it -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (HELIX-785) Report helix latency instead of user latency during top state handoff
[ https://issues.apache.org/jira/browse/HELIX-785?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16673635#comment-16673635 ] ASF GitHub Bot commented on HELIX-785: -- GitHub user zhan849 opened a pull request: https://github.com/apache/helix/pull/292 [HELIX-785] Record helix latency instead of user latency in top state handoff metrics - top state handoff reports helix latency instead of user latency - modified test cases You can merge this pull request into a Git repository by running: $ git pull https://github.com/zhan849/helix harry/top-state-handoff-metrics Alternatively you can review and apply these changes as the patch at: https://github.com/apache/helix/pull/292.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #292 commit 37a58cfff91fb5f6608a4a06d1922bb5a5eb9ca1 Author: Harry Zhang Date: 2018-11-02T18:30:15Z [HELIX-785] Record helix latency instead of user latency in top state handoff metrics > Report helix latency instead of user latency during top state handoff > - > > Key: HELIX-785 > URL: https://issues.apache.org/jira/browse/HELIX-785 > Project: Apache Helix > Issue Type: Task >Reporter: Harry Zhang >Assignee: Harry Zhang >Priority: Major > > Currently we are reporting top state handoff user latency, but we should > report Helix latency instead. user should have their way of monitoring their > own state transitions. > AC: > 1. Implement reporting Helix latency for top state handoff and test it -- This message was sent by Atlassian JIRA (v7.6.3#76005)