If the intent was to remove support from trunk/3.0, then I think all traces of it should have been removed from the codebase. Right now, I see that FsPermission#DEPRECATED_UMASK_LABEL is still present. If that had been removed, then it also would have forced us to fix the test failures reported in
HDFS-9451. Those test suites would have started getting compilation failures after removal of FsPermission#DEPRECATED_UMASK_LABEL.
I propose that we revert the current patch, create a new patch that really removes all traces from the codebase, and incorporates any test fixes currently tracked in
HDFS-9451. Please let me know if I'm missing some reason that FsPermission#DEPRECATED_UMASK_LABEL needs to remain. Thanks!