Refactor CommonWorkers.java. Lots of code is repetitive and can be factored out into shared private methods. This includes things like:

• Exception handling
• Queries
• Building the return data structures

Many problems exist in this file:

• Commented out code should be removed
• File is too big, need to be broken down
• Ugly business logic in most of the functions which can be simplified considerably
• Delegator code needs to be isolated and side effects kept down to a minimum

org.ofbiz.entity.util.EntitySaxReader requires refactoring to achieve the following goals

• Refactor EntitySaxReader to an interface
• Create an implementing class
• Remove all dependencies on Javolution
• Fix all dependencies in the framework to match the new signature of the interface
• Delete the javolution library from the framework

Add JavaDoc comments to each method that is refactored. "if you are in a hole, the first step is to stop digging".

• If people are going to the trouble to read and reverse engineer the code, it makes sense to use that knowledge to add comments.
• Having a written contract will improve the robustness of the framework but helping to avoid changes that affect the contract.
• It will also make it easier for new contributors to get confident about using the code or even modifying methods or extending the class.