Skip to content

#SPR-17286 fix proposal #1962

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

#SPR-17286 fix proposal #1962

wants to merge 1 commit into from

Conversation

Emmenemoi
Copy link

@pivotal-issuemaster
Copy link

@Emmenemoi Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@Emmenemoi Thank you for signing the Contributor License Agreement!

@jhoeller
Copy link
Contributor

As per my comment on JIRA, I went with CopyOnWriteArrayList instead, also for the embeddedValueResolvers field, and covered hasInstantiationAwareBeanPostProcessors / hasDestructionAwareBeanPostProcessors as volatile fields that are guaranteed to be visible before a newly added post-processor instance is exposed. Thanks for the pull request, in any case!

@jhoeller jhoeller closed this Sep 18, 2018
@spring-projects-issues spring-projects-issues mentioned this pull request Jan 11, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Emmenemoi @pivotal-issuemaster @jhoeller