[ https://issues.apache.org/jira/browse/YARN-7102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16240578#comment-16240578 ]
Jason Lowe commented on YARN-7102: ---------------------------------- Thanks for updating the patch! The new RMNode#setLastNodeHeartBeatResponse method isn't appropriate, IMO. This is only called by RMNodeImpl which implies this is an implementation detail of RMNodeImpl and not something that needs to be or should be exposed in the RMNode interface. As I mentioned above, we don't need a method at all, as the reconnect transition code can simply do: {{newNode.latestNodeHeartBeatResponse = rmNode.getLastNodeHeartbeatResponse()}}. If desired there can be a private method on the RMNodeImpl that the transition code can call, but this is all implementation detail that should not impact the RMNode interface. > NM heartbeat stuck when responseId overflows MAX_INT > ---------------------------------------------------- > > Key: YARN-7102 > URL: https://issues.apache.org/jira/browse/YARN-7102 > Project: Hadoop YARN > Issue Type: Bug > Reporter: Botong Huang > Assignee: Botong Huang > Priority: Critical > Attachments: YARN-7102-branch-2.8.v10.patch, > YARN-7102-branch-2.8.v11.patch, YARN-7102-branch-2.8.v9.patch, > YARN-7102-branch-2.v9.patch, YARN-7102-branch-2.v9.patch, > YARN-7102-branch-2.v9.patch, YARN-7102.v1.patch, YARN-7102.v12.patch, > YARN-7102.v13.patch, YARN-7102.v14.patch, YARN-7102.v2.patch, > YARN-7102.v3.patch, YARN-7102.v4.patch, YARN-7102.v5.patch, > YARN-7102.v6.patch, YARN-7102.v7.patch, YARN-7102.v8.patch, YARN-7102.v9.patch > > > ResponseId overflow problem in NM-RM heartbeat. This is same as AM-RM > heartbeat in YARN-6640, please refer to YARN-6640 for details. -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org