[GitHub] [hadoop] Neilxzn commented on pull request #2719: YARN-10649. fix RMNodeImpl.updateExistContainers leak

2021-03-01 Thread GitBox
Neilxzn commented on pull request #2719: URL: https://github.com/apache/hadoop/pull/2719#issuecomment-788551894 @bibinchundatt Can you please take a look at this PR? thx This is an automated message from the Apache Git Servic

[GitHub] [hadoop] Neilxzn commented on pull request #2719: YARN-10649. fix RMNodeImpl.updateExistContainers leak

2021-02-25 Thread GitBox
Neilxzn commented on pull request #2719: URL: https://github.com/apache/hadoop/pull/2719#issuecomment-785766859 @aajisaka Hi, does this patch require other improvements? Thank you for your review. This is an automated me

[GitHub] [hadoop] Neilxzn commented on pull request #2719: YARN-10649. fix RMNodeImpl.updateExistContainers leak

2021-02-24 Thread GitBox
Neilxzn commented on pull request #2719: URL: https://github.com/apache/hadoop/pull/2719#issuecomment-785600706 > There is no regression test, so the error occurred. modify TestRMNodeTransitions.testDisappearingContainer --

[GitHub] [hadoop] Neilxzn commented on pull request #2719: YARN-10649. fix RMNodeImpl.updateExistContainers leak

2021-02-24 Thread GitBox
Neilxzn commented on pull request #2719: URL: https://github.com/apache/hadoop/pull/2719#issuecomment-785181543 cc @jojochuang Can you help me review this patch?And it failed to build on jenkins but I can't find any errors about it.

[GitHub] [hadoop] Neilxzn commented on pull request #2719: YARN-10649. fix RMNodeImpl.updateExistContainers leak

2021-02-24 Thread GitBox
Neilxzn commented on pull request #2719: URL: https://github.com/apache/hadoop/pull/2719#issuecomment-784905670 @bibinchundatt @macroadster Can you please review this patch? thx This is an automated message from the Apache G

[GitHub] [hadoop] Neilxzn commented on pull request #2719: YARN-10649. fix RMNodeImpl.updateExistContainers leak

2021-02-23 Thread GitBox
Neilxzn commented on pull request #2719: URL: https://github.com/apache/hadoop/pull/2719#issuecomment-784854345 @bibinchundatt Can you please review this patch? thx This is an automated message from the Apache Git Service. To