[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14266255#comment-14266255 ] Hudson commented on YARN-2958: -- FAILURE: Integrated in Hadoop-Mapreduce-trunk #2016 (See [htt

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14266212#comment-14266212 ] Hudson commented on YARN-2958: -- FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #66 (See

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14266171#comment-14266171 ] Hudson commented on YARN-2958: -- SUCCESS: Integrated in Hadoop-Hdfs-trunk-Java8 #62 (See [http

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14266148#comment-14266148 ] Hudson commented on YARN-2958: -- FAILURE: Integrated in Hadoop-Hdfs-trunk #1997 (See [https://

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14265997#comment-14265997 ] Hudson commented on YARN-2958: -- FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #65 (See [http

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14265982#comment-14265982 ] Hudson commented on YARN-2958: -- FAILURE: Integrated in Hadoop-Yarn-trunk #799 (See [https://b

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14265409#comment-14265409 ] Varun Saxena commented on YARN-2958: Thanks [~jianhe] and [~zjshen] for the review. >

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14265154#comment-14265154 ] Hudson commented on YARN-2958: -- FAILURE: Integrated in Hadoop-trunk-Commit #6808 (See [https:

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14265118#comment-14265118 ] Jian He commented on YARN-2958: --- lgtm, thanks [~varun_saxena] and [~zjshen] ! > RMStateStore

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14265043#comment-14265043 ] Hadoop QA commented on YARN-2958: - {color:red}-1 overall{color}. Here are the results of t

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14265018#comment-14265018 ] Zhijie Shen commented on YARN-2958: --- The last patch looks good to me. [~jianhe], do you h

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14264831#comment-14264831 ] Varun Saxena commented on YARN-2958: bq. If you take a look at the old addStoreOrUpdate

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14264823#comment-14264823 ] Zhijie Shen commented on YARN-2958: --- bq. And if it doesnt exist we store it as a new toke

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14263602#comment-14263602 ] Varun Saxena commented on YARN-2958: [~zjshen], thanks for the review. Please find my r

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14263289#comment-14263289 ] Zhijie Shen commented on YARN-2958: --- 1. No need to add {{isUpdateSeqNo}}. Updating a non-

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14263241#comment-14263241 ] Zhijie Shen commented on YARN-2958: --- Will review the patch > RMStateStore seems to unnec

[jira] [Commented] (YARN-2958) RMStateStore seems to unnecessarily and wrongly store sequence number separately

[ https://issues.apache.org/jira/browse/YARN-2958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14261385#comment-14261385 ] Varun Saxena commented on YARN-2958: [~jianhe] / [~zjshen], kindly review > RMStateSto