[ https://issues.apache.org/jira/browse/MAPREDUCE-4822?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Chu Tong updated MAPREDUCE-4822: -------------------------------- Status: Patch Available (was: Open) > Unnecessary conversions in History Events > ----------------------------------------- > > Key: MAPREDUCE-4822 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-4822 > Project: Hadoop Map/Reduce > Issue Type: Improvement > Components: jobhistoryserver > Affects Versions: 0.23.4 > Reporter: Robert Joseph Evans > Assignee: Chu Tong > Priority: Trivial > Labels: patch > Fix For: 0.24.0 > > Attachments: MAPREDUCE-4822.patch > > > There are a number of conversions in the Job History Event classes that are > totally unnecessary. It appears that they were originally used to convert > from the internal avro format, but now many of them do not pull the values > from the avro they store them internally. > For example: > {code:title=TaskAttemptFinishedEvent.java} > /** Get the task type */ > public TaskType getTaskType() { > return TaskType.valueOf(taskType.toString()); > } > {code} > The code currently is taking an enum, converting it to a string and then > asking the same enum to convert it back to an enum. If java work properly > this should be a noop and a reference to the original taskType should be > returned. > There are several places that a string is having toString called on it, and > since strings are immutable it returns a reference to itself. > The various ids are not immutable and probably should not be changed at this > point. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira