Ashutosh Gupta

My bad, Please do it whenever you have some free cycles. Thanks.

Thanks @pjfanning. I can help in reviewing. My only concern here is that - As there is no security related issue or any major bug encountered in the current version....

Thanks @pjfanning for more explanation about this upgrade. Given >Current version 2.1.1 has no CVEs but all higher versions have CVEs except for the latest release 2.8.0. Still feels like...

> hadoop.yarn.server.resourcemanager.reservation.TestCapacityOverTimePolicy Failure is unrelated, Ran it multiple times in local - its working fine.

@ayushtkn - I have addressed your comments, can you please review it again. Thanks.

Hi @ayushtkn - Thanks for your review and approval, I have update the log line as well. Can you commit this

Thanks @aajisaka for final review/merge and @saswata-dutta for your additional review.

@jojochuang I ran the following tests which are failing in Yetus Run and they are passing in local. ``` hadoop.mapred.TestMRTimelineEventHandling hadoop.mapred.TestMiniMRClasspath hadoop.yarn.client.api.impl.TestAMRMClient hadoop.hdfs.server.datanode.TestDataNodeRollingUpgrade hadoop.hdfs.tools.TestDFSAdmin hadoop.hdfs.server.datanode.fsdataset.impl.TestLazyPersistReplicaRecovery ```

No issues. You are most welcome.

Changes makes sense. +1 (non-binding)