Skip to content
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

Clear annotation defaults cache to prevent pollution from one language to another #6734

Merged
merged 4 commits into from Jan 11, 2022

Conversation

jameskleeh
Copy link
Contributor

This also reorders the reference classes to register annotation defaults before creating annotation metadata because the creation references the defaults.

@dstepanov
Copy link
Contributor

I have checked this PR fixes errors in Micronaut Data

@graemerocher
Copy link
Contributor

fails a lot of tests though :/

@jameskleeh jameskleeh merged commit b0e48f9 into 3.2.x Jan 11, 2022
@jameskleeh jameskleeh deleted the default_cache branch January 11, 2022 19:51
@dstepanov
Copy link
Contributor

Looks like it fixed some compile errors but then 3.2.5 has introduced other problems. Cases with ap.getRequiredValue(AutoPopulated.UPDATEABLE, Boolean.class) fail on required value and if I rewrite them ./gradlew clean :data-jdbc:test will fail on missing compilation info at the runtime.

https://github.com/micronaut-projects/micronaut-data/compare/err

@graemerocher
Copy link
Contributor

Hmm I guess the changes added between when you verified the PR and it was merged caused the issues

@dstepanov
Copy link
Contributor

Actually, I have only checked the :data-processor:test expecting that runtime will be fine

yawkat pushed a commit that referenced this pull request Jan 21, 2022
…e to another (#6734)

* Clear annotation defaults cache to prevent pollution from one language to another

* Separate runtime and compile time caches

* Copy defaults to runtime for Groovy tests

* Use a regular map
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants