So looks like this will return a new desc factory object each time it's invoked. Is that needed? 

Because, that means, in the hRaven runs, for each job history file, a new job desc factory object would get created each time. I think let's not do that.  Is this an option: can we keep the private static final lines that are being initialized above and only return the appropriate object in this switch statement?