Bug #2716

very long Groovy compilation time when launching import modules for the first time in a session

Added by Matthieu Decorde about 2 years ago. Updated 8 months ago.

Status:New Start date:01/16/2020
Priority:Urgent Due date:
Assignee:- % Done:

80%

Category:Macros Spent time: -
Target version:TXM 0.8.2

Description

Currently there is a very long compilation time before running any Groovy script (at first run or after editing a script).

Solution 0

The GroovyScriptEngine classloader needs to access all ClassLoaders to find out unknown classes (e.g a TXM extension classes like the "URSCorpora" class)

reduce the number of tested ClassLoader when the GroovyScriptEngine loads a class :
  • only use TXM plugin loaders -> ~ 70 loaders
  • only use the "*.rcp" plugin loaders (they contains the "*.core" plugin loaders) -> 34 loaders
  • test the EclipseClassLoader name against the loaded class package name
    • first speed tests were not good

Related issues

related to Bug #3080: very long Groovy compilation time when launching import m... New 01/16/2020

Associated revisions

Revision 3004
Added by Matthieu Decorde 12 months ago

less ClassLoader to check when calling Groovy scripts refs #2716

History

#1 Updated by Matthieu Decorde over 1 year ago

  • Subject changed from RCP: X.X, pre-compile Groovy scripts to very long Groovy compilation time when launching import modules for the first time in a session
  • Description updated (diff)

#2 Updated by Matthieu Decorde over 1 year ago

  • Description updated (diff)

#3 Updated by Matthieu Decorde over 1 year ago

  • Priority changed from Normal to High

#4 Updated by Matthieu Decorde 12 months ago

  • Priority changed from High to Urgent

#5 Updated by Matthieu Decorde 12 months ago

  • Description updated (diff)

#6 Updated by Matthieu Decorde 12 months ago

  • Description updated (diff)

#7 Updated by Matthieu Decorde 12 months ago

  • Description updated (diff)

#8 Updated by Matthieu Decorde 8 months ago

  • Description updated (diff)

#9 Updated by Matthieu Decorde 8 months ago

  • Description updated (diff)

#10 Updated by Matthieu Decorde 8 months ago

  • Description updated (diff)
  • % Done changed from 0 to 80

Also available in: Atom PDF