You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@openjpa.apache.org by GitBox <gi...@apache.org> on 2020/07/14 06:23:38 UTC

[GitHub] [openjpa] struberg commented on pull request #64: [2.2.x] OPENJPA-2817: OpenJPA enhancer needs improved reentrancy

struberg commented on pull request #64:
URL: https://github.com/apache/openjpa/pull/64#issuecomment-657993792


   I've now read the SE7 documentation about a few times and I think the ThreadLocal<Boolean> is not enough for this specific case. From the ClassLoader doc it appears that the lock now got moved from a full exclusive Semaphore to a Lock by ClassLoader+Classname.
   So - without digging into all the details - it seems logical to also move from a Boolean to a Map<String, Boolean> were the key is the CL id + Classname ?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org