[jira] [Commented] (YARN-6112) fsOpDurations.addUpdateCallDuration() should be independent to LOG level

2017-02-07 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15857219#comment-15857219 ] Hadoop QA commented on YARN-6112: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-6112) fsOpDurations.addUpdateCallDuration() should be independent to LOG level

2017-02-07 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15857137#comment-15857137 ] Karthik Kambatla commented on YARN-6112: +1 for the branch-2 patch too, pending Jenkins. >

[jira] [Commented] (YARN-6112) fsOpDurations.addUpdateCallDuration() should be independent to LOG level

2017-02-07 Thread Yufei Gu (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15856977#comment-15856977 ] Yufei Gu commented on YARN-6112: [~kasha], thanks for the review. Uploaded the patch for branch-2. >

[jira] [Commented] (YARN-6112) fsOpDurations.addUpdateCallDuration() should be independent to LOG level

2017-02-07 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15856930#comment-15856930 ] Karthik Kambatla commented on YARN-6112: +1 on the trunk patch. [~yufeigu] - can you post another

[jira] [Commented] (YARN-6112) fsOpDurations.addUpdateCallDuration() should be independent to LOG level

2017-01-26 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840753#comment-15840753 ] Hadoop QA commented on YARN-6112: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-6112) fsOpDurations.addUpdateCallDuration() should be independent to LOG level

2017-01-26 Thread Yufei Gu (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840655#comment-15840655 ] Yufei Gu commented on YARN-6112: Patch 003: Remove the updateCall metrics and related measurement in

[jira] [Commented] (YARN-6112) fsOpDurations.addUpdateCallDuration() should be independent to LOG level

2017-01-26 Thread Yufei Gu (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840585#comment-15840585 ] Yufei Gu commented on YARN-6112: There is an another duration measurement outside of the function update()

[jira] [Commented] (YARN-6112) fsOpDurations.addUpdateCallDuration() should be independent to LOG level

2017-01-26 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840543#comment-15840543 ] Karthik Kambatla commented on YARN-6112: Sorry for screwing it up. Absolutely, it is a mistake.

[jira] [Commented] (YARN-6112) fsOpDurations.addUpdateCallDuration() should be independent to LOG level

2017-01-25 Thread Daniel Templeton (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15838762#comment-15838762 ] Daniel Templeton commented on YARN-6112: If [~kasha] agrees that change is a mistake, I'm +1 for

[jira] [Commented] (YARN-6112) fsOpDurations.addUpdateCallDuration() should be independent to LOG level

2017-01-25 Thread Daniel Templeton (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15838759#comment-15838759 ] Daniel Templeton commented on YARN-6112: [~kasha], the issue is that the call to