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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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