[GitHub] [hadoop] virajjasani commented on pull request #4027: YARN-11081. TestYarnConfigurationFields consistently keeps failing

2022-03-04 Thread GitBox
virajjasani commented on pull request #4027: URL: https://github.com/apache/hadoop/pull/4027#issuecomment-1059105858 > We should not add any parameter to configurationPrefixToSkipCompare without some specific reason (e.g., deprecated parameter). I see, the configs that I saw were dep

[GitHub] [hadoop] virajjasani commented on pull request #4027: YARN-11081. TestYarnConfigurationFields consistently keeps failing

2022-03-03 Thread GitBox
virajjasani commented on pull request #4027: URL: https://github.com/apache/hadoop/pull/4027#issuecomment-1058138750 Thanks @aajisaka. Based on the ApplicationPlacementAllocatorFactory, it looks like the code has sufficient null check to handle missing property value: ``` try {