Uploaded image for project: 'Spring Framework'
  1. Spring Framework
  2. SPR-13655

AbstractAutoProxyCreator.getCacheKey generate lots of String garbage

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Complete
    • 4.2.2
    • 4.2.3
    • Core:AOP
    • true

    Description

      Current implementation of getCacheKey uses string concatenation: https://github.com/spring-projects/spring-framework/blob/master/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/AbstractAutoProxyCreator.java#L306-L308

      This leads to two problems:
      1) Excessive heap garbage due to throw-away string creation
      2) Excessive CPU due to computing hash code for the newly created strings.

      In our spring context present 1200+ beans and ~20 beanPostProcessors as result E2E spenario for 30 minute generate ~300GB char[]

      It our problem that legacy part of system use BeanFactory.getBean instead of inject dependency correctly. But problem described above also affect application start time because spring for inject dependency also do getBean by Class and as result spring for each class iterate by all beans in context and generate cache key like:

      	protected Object getCacheKey(Class<?> beanClass, String beanName) {
      		return beanClass.getName() + "_" + beanName;
      	}
      

      Attachments

        Issue Links

          Activity

            People

              juergen.hoeller Juergen Hoeller
              gordiychuk Vladimir Gordiychuk
              Spring Issues Spring Issues
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                3 years, 47 weeks, 2 days ago