[jira] [Commented] (HELIX-785) Report helix latency instead of user latency during top state handoff

2018-11-02 Thread Hudson (JIRA)


[ 
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

2018-11-02 Thread ASF GitHub Bot (JIRA)


[ 
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

2018-11-02 Thread ASF GitHub Bot (JIRA)


[ 
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)