[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2017-08-29 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16146725#comment-16146725 ] Hudson commented on YARN-5922: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #1227

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-12-08 Thread Haibo Chen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15733554#comment-15733554 ] Haibo Chen commented on YARN-5922: -- Thanks [~sjlee0] for your reviews! > Remove direct re

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-12-08 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15733392#comment-15733392 ] Hudson commented on YARN-5922: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #1097

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-12-08 Thread Sangjin Lee (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15733303#comment-15733303 ] Sangjin Lee commented on YARN-5922: --- LGTM. Committing shortly. > Remove direct reference

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

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

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-12-07 Thread Sangjin Lee (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15731209#comment-15731209 ] Sangjin Lee commented on YARN-5922: --- Thanks [~haibochen]. Can you address the lone checks

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

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

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

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

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-12-06 Thread Sangjin Lee (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15727426#comment-15727426 ] Sangjin Lee commented on YARN-5922: --- I cherry-picked that commit into YARN-5355 and YARN-

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

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

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-12-06 Thread Haibo Chen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15726932#comment-15726932 ] Haibo Chen commented on YARN-5922: -- Uploaded a new patch for trunk branch now that HADOOP-

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-12-02 Thread Haibo Chen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15716056#comment-15716056 ] Haibo Chen commented on YARN-5922: -- I see. I have uploaded a patch in HADOOP-13859 with th

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-12-02 Thread Sangjin Lee (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15715917#comment-15715917 ] Sangjin Lee commented on YARN-5922: --- Yes, I see that. That's a little different in that i

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-12-02 Thread Haibo Chen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15715898#comment-15715898 ] Haibo Chen commented on YARN-5922: -- Agreed. That's why I made the change in TestConfigurat

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-12-02 Thread Sangjin Lee (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15715811#comment-15715811 ] Sangjin Lee commented on YARN-5922: --- That said, it sounds bit silly that people need to w

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-12-01 Thread Sangjin Lee (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15713815#comment-15713815 ] Sangjin Lee commented on YARN-5922: --- Sorry I was assuming incorrectly that these properti

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-12-01 Thread Haibo Chen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15712547#comment-15712547 ] Haibo Chen commented on YARN-5922: -- Thanks [~sjlee0] for your review! Does it make sense t

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-11-30 Thread Sangjin Lee (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15710920#comment-15710920 ] Sangjin Lee commented on YARN-5922: --- Thanks [~haibochen] for the updated patch! Actually

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

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

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

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

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

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

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-11-22 Thread Sangjin Lee (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15687605#comment-15687605 ] Sangjin Lee commented on YARN-5922: --- Please remove the erroneous patch (yarn5922.001.yarn

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-11-22 Thread Haibo Chen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15687591#comment-15687591 ] Haibo Chen commented on YARN-5922: -- Thanks for pointing it out [~sjlee0]. Will upload a ne

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

2016-11-22 Thread Sangjin Lee (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15687582#comment-15687582 ] Sangjin Lee commented on YARN-5922: --- [~haibochen], you'd need to name the patch files dif

[jira] [Commented] (YARN-5922) Remove direct references of HBaseTimelineWriter/Reader in core ATS classes

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