-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HHH-18901 AnnotationFormatError: Duplicate annotation for class: interface org.hibernate.bytecode.enhance.spi.EnhancementInfo #9636
base: 6.6
Are you sure you want to change the base?
Conversation
Thanks for your pull request! This pull request appears to follow the contribution rules. › This message was automatically generated. |
d4e96fc
to
43ef237
Compare
...rnate-core/src/main/java/org/hibernate/bytecode/enhance/internal/bytebuddy/EnhancerImpl.java
Show resolved
Hide resolved
.../main/java/org/hibernate/bytecode/enhance/internal/bytebuddy/OverridingClassFileLocator.java
Outdated
Show resolved
Hide resolved
...-core/src/main/java/org/hibernate/bytecode/enhance/internal/bytebuddy/SafeCacheProvider.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well this must have been quite an adventure :) Thanks a lot for taking care of it!
I added a few comments, but this looks reasonable to merge as is.
...rnate-core/src/main/java/org/hibernate/bytecode/enhance/internal/bytebuddy/EnhancerImpl.java
Outdated
Show resolved
Hide resolved
...nate-core/src/main/java/org/hibernate/bytecode/enhance/internal/bytebuddy/ModelTypePool.java
Show resolved
Hide resolved
...nate-core/src/main/java/org/hibernate/bytecode/enhance/internal/bytebuddy/ModelTypePool.java
Show resolved
Hide resolved
hibernate-core/src/main/java/org/hibernate/cfg/BytecodeSettings.java
Outdated
Show resolved
Hide resolved
...ate-core/src/main/java/org/hibernate/jpa/internal/enhance/EnhancingClassTransformerImpl.java
Outdated
Show resolved
Hide resolved
...ate-core/src/main/java/org/hibernate/jpa/internal/enhance/EnhancingClassTransformerImpl.java
Outdated
Show resolved
Hide resolved
43ef237
to
7158074
Compare
Revised and ready |
LGTM but checkstyle is not happy :) https://ge.hibernate.org/s/bcwkd7evrmxcy/failure#1 |
…rrent enhancement In particular, made it defensive against concurrent enhancement of the same resource, or of resources that might trigger loading symbols which another thread is re-processing.
7158074
to
7cd283f
Compare
Thanks - pushed a revised version |
Draft solution for https://hibernate.atlassian.net/browse/HHH-18901
We don't have a good test for this, and I didn't yet link the commits to appropriate Jiras... but I'd consider it now stable enough for others to have a look.
Specifically targeting branch 6.6 for now as that's where we can reproduce it: I'll "forward port" this if you all like it.
Thanks!