[incubator-kie-issues-1084] User eager fetching in ProcessDefinitionEntity to avoid transaction errors

[martinweiler]

Fixes apache/incubator-kie-issues#1084

[elguardian]

I am not sure this is the best approach.
@fjtirado This happens when toString is invoked as it requires to load those lazy fields. If you have any other suggestion plz. go ahead.

[fjtirado]

@martinweiler I think removing the three collections from toString method will be enough. Lets keep loading of collections lazy.
See https://struberg.wordpress.com/2016/10/15/tostring-equals-and-hashcode-in-jpa-entities/ for more rationale on that

[fjtirado]

I think removing roles and addons from toString will do the trick
Im wondering if to string should just print the id and the version (the composite key) of the process definition to avoid further problem. That will be enough for logging

[martinweiler]

@fjtirado ok, let's just remove the fields that are lazy loaded from the toString method