[jira] [Commented] (YARN-9624) Use switch case for ProtoUtils#convertFromProtoFormat containerState

2020-02-09 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/YARN-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17033212#comment-17033212 ] Hudson commented on YARN-9624: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #17932 (See

[jira] [Commented] (YARN-9624) Use switch case for ProtoUtils#convertFromProtoFormat containerState

2020-01-29 Thread Ayush Saxena (Jira)
[ https://issues.apache.org/jira/browse/YARN-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17026427#comment-17026427 ] Ayush Saxena commented on YARN-9624: +1 > Use switch case for ProtoUtils#convertFromProtoFormat

[jira] [Commented] (YARN-9624) Use switch case for ProtoUtils#convertFromProtoFormat containerState

2020-01-29 Thread Hadoop QA (Jira)
[ https://issues.apache.org/jira/browse/YARN-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17025852#comment-17025852 ] Hadoop QA commented on YARN-9624: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-9624) Use switch case for ProtoUtils#convertFromProtoFormat containerState

2020-01-07 Thread Bibin Chundatt (Jira)
[ https://issues.apache.org/jira/browse/YARN-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17009536#comment-17009536 ] Bibin Chundatt commented on YARN-9624: -- [~BilwaST] Could you update the patch ? > Use switch case

[jira] [Commented] (YARN-9624) Use switch case for ProtoUtils#convertFromProtoFormat containerState

2019-10-23 Thread Ayush Saxena (Jira)
[ https://issues.apache.org/jira/browse/YARN-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16958057#comment-16958057 ] Ayush Saxena commented on YARN-9624: For the test instead of having if-else checks and explicitly

[jira] [Commented] (YARN-9624) Use switch case for ProtoUtils#convertFromProtoFormat containerState

2019-10-23 Thread Hadoop QA (Jira)
[ https://issues.apache.org/jira/browse/YARN-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16957739#comment-16957739 ] Hadoop QA commented on YARN-9624: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-9624) Use switch case for ProtoUtils#convertFromProtoFormat containerState

2019-10-23 Thread Bilwa S T (Jira)
[ https://issues.apache.org/jira/browse/YARN-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16957680#comment-16957680 ] Bilwa S T commented on YARN-9624: - Hi [~bibinchundatt] Updated patch . Please review. > Use switch case

[jira] [Commented] (YARN-9624) Use switch case for ProtoUtils#convertFromProtoFormat containerState

2019-10-17 Thread Bibin Chundatt (Jira)
[ https://issues.apache.org/jira/browse/YARN-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16954285#comment-16954285 ] Bibin Chundatt commented on YARN-9624: -- [~BilwaST] Could you please update the patch . > Use switch

[jira] [Commented] (YARN-9624) Use switch case for ProtoUtils#convertFromProtoFormat containerState

2019-10-04 Thread Hadoop QA (Jira)
[ https://issues.apache.org/jira/browse/YARN-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16944349#comment-16944349 ] Hadoop QA commented on YARN-9624: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-9624) Use switch case for ProtoUtils#convertFromProtoFormat containerState

2019-10-04 Thread Bibin Chundatt (Jira)
[ https://issues.apache.org/jira/browse/YARN-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16944333#comment-16944333 ] Bibin Chundatt commented on YARN-9624: -- Thank you  [~BilwaST] for patch Few comments: * Remove