-
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
Properly cache ContextResolver usage for ObjectMapper #36126
Conversation
We might also want to backport the first commit to 3.2 - remains to be seen. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
I don't see how my change could be impacting this:
|
Oh my, it happens consistently on all builds. Well... that's worrying. |
it can be an image problem... |
Just checked, and the Oracle image was last updated on April 7th: 07f30aa#diff-150bdf617c2dc80dc5a9d5f1313f44a73950a96ca7b166b79a35b15f4c84f193L97-R97 |
Ah, but it's a floating tag and it's been updated 20 hours ago: https://hub.docker.com/layers/gvenzl/oracle-free/23-slim-faststart/images/sha256-44416f0de8bb6ce3ec82fb8b1ad766c4667073b723d1f02451e0f0375c0ed064?context=explore |
yup... |
I'll disable the test until they fix that image... I cannot find any older version we can use. |
There's been this report, though it doesn't really look related: gvenzl/oci-oracle-free#22 @geoand you can try changing |
Testing now with 23.2... not. |
Related: apache/camel-quarkus#5353 |
Thanks folks! |
✔️ 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. |
@geoand should this be backported to 3.4? I'm on the verge... |
Yeah, I think it makes sense |
Fixes: #36067