Uploaded image for project: 'Groovy'
  1. Groovy
  2. GROOVY-5428

Performance problem: ClassCastExceptions are created in normal execution flow which kills performance

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.8.6
    • 2.0-beta-3, 1.8.7
    • None
    • None
    • Grails 2.0.3 + Groovy 1.8.6

    Description

      This problem is a major performance bottleneck in Grails 2.0 applications.
      Currently this problem shows up when resources plugin has been installed and used in a Grails app. (see attached screenshot)

      Exceptions are relatively heavy weight in Java. Filling the stacktrace takes most of the time. Normal execution flow should never throw exceptions.
      (it is possible to disable filling the stacktrace like has been done for MissingMethodExceptionNoStack, in that case the performance overhead is usually acceptable.)

      The problem shows up whenever there is 2 classes sharing the same base class and when the other class is a Java class and the other one is a Groovy class (GroovyObject).

      I was able to reproduce the problem with this code. Run this code in a debugger and make a breakpoint for the construction of ClassCastException.
      In this example the base class is java.util.Map , the Java class is LinkedHashMap and the Groovy class is MyMap (extends LinkedHashMap).

      class ClassCastProblem {
          def doCalls() {
              def m1=[abc:'test']
              def m2=new MyMap()
              doSomething(m1)
              doSomething(m2)
              doSomething(m1)
              doSomething(m2)
              doSomething(m1)
          }
          
          void doSomething(m) {
              m.remove('test')
          }
      
          public static void main(String[] args) {
              def ccp=new ClassCastProblem()
              ccp.doCalls()
          }
      }
      
      class MyMap extends LinkedHashMap {
          
      }
      

      In a Grails app the 2 different java.util.Map implementations usually triggering this problem are LinkedHashMap and GrailsParameterMap (GroovyObject).

      In Grails core I've worked around some of the problem with this type of hack:
      https://github.com/grails/grails-core/commit/62626921ebd80e3cdeb9776fd79be7eb92f88763
      At that time I didn't have time to investigate the reason that caused the problem and just made some changes until the problem went away.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            blackdrag Jochen Theodorou
            lhotari Lari Hotari
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment