[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-02-04 Thread Chris Nauroth (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13570610#comment-13570610 ] Chris Nauroth commented on HADOOP-9252: --- Regarding the -1 from Jenkins on javac war

[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-02-04 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13570607#comment-13570607 ] Hadoop QA commented on HADOOP-9252: --- {color:red}-1 overall{color}. Here are the result

[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-02-04 Thread Chris Nauroth (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13570585#comment-13570585 ] Chris Nauroth commented on HADOOP-9252: --- +1 for this version of the patch Thanks s

[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-02-03 Thread Chris Nauroth (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13569977#comment-13569977 ] Chris Nauroth commented on HADOOP-9252: --- The changes look great. I applied the pat

[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-02-03 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13569955#comment-13569955 ] Hadoop QA commented on HADOOP-9252: --- {color:red}-1 overall{color}. Here are the result

[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-02-03 Thread Chris Nauroth (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13569845#comment-13569845 ] Chris Nauroth commented on HADOOP-9252: --- Thanks, Nicholas. It sounds like a good p

[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-02-02 Thread Tsz Wo (Nicholas), SZE (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13569626#comment-13569626 ] Tsz Wo (Nicholas), SZE commented on HADOOP-9252: Chris, thanks for checki

[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-02-01 Thread Chris Nauroth (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13569449#comment-13569449 ] Chris Nauroth commented on HADOOP-9252: --- Thanks, Nicholas. I should have looked fo

[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-02-01 Thread Tsz Wo (Nicholas), SZE (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13569359#comment-13569359 ] Tsz Wo (Nicholas), SZE commented on HADOOP-9252: Chris, thanks for the de

[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-02-01 Thread Chris Nauroth (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13569335#comment-13569335 ] Chris Nauroth commented on HADOOP-9252: --- Hi Nicholas, I'm assuming that for backwa

[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-01-28 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13564607#comment-13564607 ] Hadoop QA commented on HADOOP-9252: --- {color:red}-1 overall{color}. Here are the result

[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-01-27 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13563971#comment-13563971 ] Hadoop QA commented on HADOOP-9252: --- {color:red}-1 overall{color}. Here are the result

[jira] [Commented] (HADOOP-9252) StringUtils.limitDecimalTo2(..) is unnecessarily synchronized

2013-01-25 Thread Tsz Wo (Nicholas), SZE (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-9252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13563287#comment-13563287 ] Tsz Wo (Nicholas), SZE commented on HADOOP-9252: Also humanReadableInt(..