[GitHub] [hadoop] zuston commented on pull request #4060: YARN-11084. Introduce new config to specify AM default node-label whe…

2022-03-24 Thread GitBox
zuston commented on pull request #4060: URL: https://github.com/apache/hadoop/pull/4060#issuecomment-1077727264 Thanks for your patience and review @9uapaw -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [hadoop] zuston commented on pull request #4060: YARN-11084. Introduce new config to specify AM default node-label whe…

2022-03-22 Thread GitBox
zuston commented on pull request #4060: URL: https://github.com/apache/hadoop/pull/4060#issuecomment-1075080288 All Done @9uapaw -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific com

[GitHub] [hadoop] zuston commented on pull request #4060: YARN-11084. Introduce new config to specify AM default node-label whe…

2022-03-21 Thread GitBox
zuston commented on pull request #4060: URL: https://github.com/apache/hadoop/pull/4060#issuecomment-1073984190 Updated. Could u help review again? @9uapaw -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [hadoop] zuston commented on pull request #4060: YARN-11084. Introduce new config to specify AM default node-label whe…

2022-03-21 Thread GitBox
zuston commented on pull request #4060: URL: https://github.com/apache/hadoop/pull/4060#issuecomment-1073944578 Gentle ping @9uapaw . Thanks ~ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [hadoop] zuston commented on pull request #4060: YARN-11084. Introduce new config to specify AM default node-label whe…

2022-03-14 Thread GitBox
zuston commented on pull request #4060: URL: https://github.com/apache/hadoop/pull/4060#issuecomment-1067549091 Updated. Please check it @9uapaw Thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above t

[GitHub] [hadoop] zuston commented on pull request #4060: YARN-11084. Introduce new config to specify AM default node-label whe…

2022-03-14 Thread GitBox
zuston commented on pull request #4060: URL: https://github.com/apache/hadoop/pull/4060#issuecomment-1067505720 Thanks for your quick review @9uapaw Agree with your comment. So let's do a priority order user-defined AM node label > AM default-node-label-expression > default g

[GitHub] [hadoop] zuston commented on pull request #4060: YARN-11084. Introduce new config to specify AM default node-label whe…

2022-03-12 Thread GitBox
zuston commented on pull request #4060: URL: https://github.com/apache/hadoop/pull/4060#issuecomment-1066046782 If u have time, could you help review it? @szilard-nemeth @9uapaw @virajjasani @jojochuang Thanks -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [hadoop] zuston commented on pull request #4060: YARN-11084. Introduce new config to specify AM default node-label whe…

2022-03-10 Thread GitBox
zuston commented on pull request #4060: URL: https://github.com/apache/hadoop/pull/4060#issuecomment-1063795860 More tests will be attached later. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to