Skip to content
This repository has been archived by the owner on Nov 26, 2024. It is now read-only.

Allowing blocking calls inside Jackson's DeserializerCache::_createAndCacheValueDeserializer resolves the IllegalMonitorStateException #769

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

softqwewasd
Copy link
Member

@softqwewasd softqwewasd commented Jun 17, 2024

Allowing blocking calls inside Jackson's DeserializerCache::_createAndCacheValueDeserializer resolves the IllegalMonitorStateException.

See FasterXML/jackson-databind#4430

This PR resolves the issue we had when upgrading to Jackson 2.17.1. The upgrade to 2.17.1 caused some of our tests to fail with IllegalMonitorStateException. By configuring Blockhound to allow blocking calls inside _createAndCacheValueDeserializer, IllegalMonitorStateException is no longer thrown. You can read more about this issue here FasterXML/jackson-databind#4430 (comment).

…dCacheValueDeserializer resolves the IllegalMonitorStateException
Copy link

Copy link
Contributor

@splitfeed splitfeed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So this is a dependency upgrade and fix of a test?

@softqwewasd
Copy link
Member Author

So this is a dependency upgrade and fix of a test?

Pretty much. I've updated the PR description with some additional context.

@softqwewasd softqwewasd merged commit e8c42bd into master Jun 19, 2024
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants