Would it make sense to turn the comparison around like so:
JobHistoryKeys.LAUNCH_TIME.toString().equals(JobHistoryKeys.HADOOP2_TO_HADOOP1_MAPPING.get(key))
or do we know for sure that JobHistoryKeys.HADOOP2_TO_HADOOP1_MAPPING.get(key) will never return null ?