Details
-
Task
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
None
-
None
-
Reviewed
Description
In both GetApplicationsRequestPBImpl and ApplicationSubmissionContextPBImpl, there is a forced lowercase conversion:
checkTags(tags);
// Convert applicationTags to lower case and add
this.applicationTags = new TreeSet<>();
for (String tag : tags) {
this.applicationTags.add(StringUtils.toLowerCase(tag));
}
}
However, we encountered some cases where this is not desirable for "userid" tags.
Proposed solution: since both classes are pretty low-level and can be often instantiated, a Configuration object which loads yarn-site.xml should be cached inside them. A new property should be created which tells whether lowercase conversion should occur or not.