-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
ArC fixes for spec compatibility, round 2 #30666
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@Ladicek I don't think I have seen this test fail before ^ |
Interesting. The failure is:
|
Note that this test used to be flaky, but I am not aware of it being problematic since we swapped tests to use RR instead of classic RE (Oct last year). |
This commit also fixes a NPE in case of static producer method declared on a `@Dependent` bean.
Previously, we always used `GenericArrayType` to represent array bean types at runtime. This is unexpected for people, and also for at least one TCK test that expects a `java.lang.Class` representation of `String[]`. This commit makes sure that arrays of primitive types and simple class types are represented using an array-typed `java.lang.Class` object.
Frankly speaking, I doubt the CP test failure has anything to do with changes in this PR and it's not reproducible locally. |
✔️ The latest workflow run for the pull request has completed successfully. It should be safe to merge provided you have a look at the other checks in the summary. |
Related to #28558
@Priority
Bean.create()
to wrap checked exceptions inCreationException
create()
is now calleddoCreate()
andcreate()
just wraps that intotry
/catch
@Dependent
beanGenericArrayType
to represent array bean types at runtime, which is unexpected for people, and also for at least one TCK test that expects ajava.lang.Class
representation ofString[]