[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2016-06-14 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15330711#comment-15330711 ] Jian He commented on YARN-1942: --- lgtm, thanks ! > Many of ConverterUtils methods need to hav

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2016-06-13 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15328924#comment-15328924 ] Wangda Tan commented on YARN-1942: -- Failed unit tests are not related. > Many of Converte

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2016-06-09 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15323922#comment-15323922 ] Jian He commented on YARN-1942: --- [~leftnoteasy], patch does not apply to branch-2, could you

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2016-06-09 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15323463#comment-15323463 ] Wangda Tan commented on YARN-1942: -- MAPREDUCE-6682 tracks TestMRCJCFileOutputCommitter fai

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2016-06-09 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15323413#comment-15323413 ] Jian He commented on YARN-1942: --- lgtm, thanks > Many of ConverterUtils methods need to have

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2016-06-09 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15322877#comment-15322877 ] Wangda Tan commented on YARN-1942: -- [~jianhe], latest Jenkins looks fine to me. Could you

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2016-06-07 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15319851#comment-15319851 ] Jian He commented on YARN-1942: --- patch looks good, are the test failures related ? > Many of

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2016-05-25 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15300894#comment-15300894 ] Jian He commented on YARN-1942: --- looks good overall, few comments - No need to create a new A

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2016-05-13 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15283272#comment-15283272 ] Jian He commented on YARN-1942: --- - setter should remain private/unstable {code} @Public

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2015-06-03 Thread Sergey Shelukhin (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14571761#comment-14571761 ] Sergey Shelukhin commented on YARN-1942: No, it's used in production code as far as

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2015-06-03 Thread Zhijie Shen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14571755#comment-14571755 ] Zhijie Shen commented on YARN-1942: --- [~sershe], would you please comment? > Many of Conv

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2015-06-03 Thread Vinod Kumar Vavilapalli (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14571668#comment-14571668 ] Vinod Kumar Vavilapalli commented on YARN-1942: --- bq. It seems that we have mo

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2015-06-03 Thread Zhijie Shen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14571623#comment-14571623 ] Zhijie Shen commented on YARN-1942: --- It seems that we have more than ConverterUtils that

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2015-05-01 Thread Xuan Gong (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14524026#comment-14524026 ] Xuan Gong commented on YARN-1942: - Thanks for the patch, [~leftnoteasy] Some comments: * ca

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2015-03-12 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14359438#comment-14359438 ] Hadoop QA commented on YARN-1942: - {color:red}-1 overall{color}. Here are the results of t

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2015-03-12 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14359169#comment-14359169 ] Hadoop QA commented on YARN-1942: - {color:red}-1 overall{color}. Here are the results of t

[jira] [Commented] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

2015-03-12 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14359091#comment-14359091 ] Wangda Tan commented on YARN-1942: -- Working on this, will post a patch soon. > Many of Co