It was noticed there are some unnecessary dependency into Directory classes.
Uploaded a patch to do the clean up.
This message was automatically generated.
Thanks, Kai Zheng. Loogs good in general. Your indentation is off in the Strings line:
- !Strings.isEmpty(rr.getResourceName()) ? rr
+ !Strings.isNullOrEmpty(rr.getResourceName()) ? rr
The original line was correctly indented.
Thanks Daniel. I manually edit it and uploaded a new version.
+1 committing it.
-1 asflicense 0m 26s Patch generated 1 ASF License warnings.
This is due to HDFS-9582
FAILURE: Integrated in Hadoop-trunk-Commit #9004 (See https://builds.apache.org/job/Hadoop-trunk-Commit/9004/)
YARN-4480. Clean up some inappropriate imports. (Kai Zheng via (umamahesh: rev 0f82b5d878a76b1626c9e07b2fbb55ce2a79232a)
Committed to trunk and branch-2, Thanks Kai
Thanks Uma for reviewing and committing this!