[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-08 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15475414#comment-15475414 ] Hadoop QA commented on HDFS-10742: -- | (/) *{color:green}+1 overall{color}* | \\ \\ || Vot

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-08 Thread Arpit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15475357#comment-15475357 ] Arpit Agarwal commented on HDFS-10742: -- +1 for the v017 patch. > Measurement of lock

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-08 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15475358#comment-15475358 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-08 Thread Erik Krogen (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15475060#comment-15475060 ] Erik Krogen commented on HDFS-10742: For v16 patch IIUC you actually still have the de

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-07 Thread Arpit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15471815#comment-15471815 ] Arpit Agarwal commented on HDFS-10742: -- Thank you [~chris.douglas]. Your patch lgtm.

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-07 Thread Chen Liang (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15471323#comment-15471323 ] Chen Liang commented on HDFS-10742: --- v015 patch looks to be a better one to me compared

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-07 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15470079#comment-15470079 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-06 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15469154#comment-15469154 ] Hadoop QA commented on HDFS-10742: -- | (/) *{color:green}+1 overall{color}* | \\ \\ || Vot

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-06 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15468936#comment-15468936 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-06 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15468745#comment-15468745 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-04 Thread Chris Douglas (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15463720#comment-15463720 ] Chris Douglas commented on HDFS-10742: -- bq. Removed per-method measurement and aggreg

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-02 Thread Arpit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15458753#comment-15458753 ] Arpit Agarwal commented on HDFS-10742: -- By the way I am +1 on the v11 patch. Will hol

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-09-02 Thread Arpit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15458717#comment-15458717 ] Arpit Agarwal commented on HDFS-10742: -- Hi [~chris.douglas], are you okay with procee

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-31 Thread Chen Liang (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15453528#comment-15453528 ] Chen Liang commented on HDFS-10742: --- Thanks [~arpitagarwal] for the review! Updated a pa

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-31 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15453065#comment-15453065 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-31 Thread Arpit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15452877#comment-15452877 ] Arpit Agarwal commented on HDFS-10742: -- Thanks [~vagarychen]. The change lgtm. A few

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-30 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15450777#comment-15450777 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-30 Thread Arpit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15449980#comment-15449980 ] Arpit Agarwal commented on HDFS-10742: -- Hi [~vagarychen], a couple of comments: # {{c

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-29 Thread Chris Douglas (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15447450#comment-15447450 ] Chris Douglas commented on HDFS-10742: -- bq. That adds a new object allocation for eve

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-29 Thread Arpit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15447205#comment-15447205 ] Arpit Agarwal commented on HDFS-10742: -- IIUC that'd be used like {code} try (AutoClo

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-29 Thread Chris Douglas (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15447021#comment-15447021 ] Chris Douglas commented on HDFS-10742: -- bq. I can't be sure other developers will pre

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-27 Thread Arpit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15441127#comment-15441127 ] Arpit Agarwal commented on HDFS-10742: -- bq. It adds a special case for a very common

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-25 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15437671#comment-15437671 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-25 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15437482#comment-15437482 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-25 Thread Chris Douglas (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15436400#comment-15436400 ] Chris Douglas commented on HDFS-10742: -- bq. there is no other advantage. It just elim

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-24 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15435626#comment-15435626 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-24 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15435442#comment-15435442 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-24 Thread Chen Liang (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15435324#comment-15435324 ] Chen Liang commented on HDFS-10742: --- Thanks [~arpitagarwal]! Updated a patch to fix sty

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-23 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15434097#comment-15434097 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-23 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15434085#comment-15434085 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-23 Thread Arpit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15434078#comment-15434078 ] Arpit Agarwal commented on HDFS-10742: -- bq. On a related note, the virtues of AutoClo

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-23 Thread Chen Liang (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15433987#comment-15433987 ] Chen Liang commented on HDFS-10742: --- Thanks for the catch [~chris.douglas]! Updated a pa

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-23 Thread Chris Douglas (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15433853#comment-15433853 ] Chris Douglas commented on HDFS-10742: -- * Is this the intended {{ConcurrentHashMap}}

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-10 Thread Chen Liang (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15416021#comment-15416021 ] Chen Liang commented on HDFS-10742: --- The failed test is unrelated. > Measurement of loc

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-09 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15414539#comment-15414539 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (HDFS-10742) Measurement of lock held time in FsDatasetImpl

2016-08-09 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15414535#comment-15414535 ] Hadoop QA commented on HDFS-10742: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote