Skip to content

SPR-11875: Performance regression for custom autowireBean calls with many properties #570

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

Closed
wants to merge 1 commit into from

Conversation

trask
Copy link

@trask trask commented Jun 27, 2014

Please see SPR-11875.

This commit reverts e1d11ec and uses ConcurrentReferenceHashMap to fix SPR-8956 memory leak instead.

Autowire benchmark https://github.com/trask/spring-autowire-benchmark is back in line with Spring 3.1.4 after this.

I have signed and agree to the terms of the SpringSource Individual Contributor License Agreement.

Revert commit e1d11ec and use
ConcurrentReferenceHashMap to fix SPR-8956 memory leak instead.

Issue: SPR-11875, SPR-8956
@candrews
Copy link
Contributor

A series of commits has been made that addresses the issue that pull requests would fix, so I think it should be closed.

@trask do you still have the Groovy bean performance problem you mentioned at https://jira.spring.io/browse/SPR-11875?focusedCommentId=105103&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-105103 ?

@snicoll
Copy link
Member

snicoll commented Aug 21, 2014

ping @jhoeller

@pivotal-issuemaster
Copy link

@trask Please sign the Contributor License Agreement!

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

See the FAQ for frequently asked questions.

@rstoyanchev rstoyanchev added the in: core Issues in core modules (aop, beans, core, context, expression) label Nov 10, 2021
@snicoll
Copy link
Member

snicoll commented Nov 19, 2021

Closing as the related issue has been fixed.

@snicoll snicoll closed this Nov 19, 2021
@snicoll snicoll added status: superseded An issue that has been superseded by another and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Nov 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: core Issues in core modules (aop, beans, core, context, expression) status: superseded An issue that has been superseded by another
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants