[3.0.0] DefaultQueryEngine pulls JDKEvaluatorFactory because of the public static field DEFAULT #342

Closed
Maia-Everett opened this Issue Feb 14, 2013 · 1 comment

Comments

Projects
None yet
2 participants
@Maia-Everett

In the line:

queryEngine = new DefaultQueryEngine(new DefaultEvaluatorFactory(
    CollQueryTemplates.DEFAULT,
    new ECJEvaluatorFactory(getClass().getClassLoader())));

DefaultQueryEngine attempts to create a DefaultEvaluatorFactory using JDKEvaluatorFactory, because the DEFAULT field is static.

Since I don't have tools.jar on the classpath, this fails.

The field should be changed to a method that lazily initializes the default instance. Otherwise it defeats the whole point of having ECJEvaluatorFactory to avoid pulling tools.jar.

@timowest

This comment has been minimized.

Show comment
Hide comment
@timowest

timowest Feb 20, 2013

Member

Released in 3.0.0.BETA2

Member

timowest commented Feb 20, 2013

Released in 3.0.0.BETA2

@timowest timowest closed this Feb 20, 2013

@timowest timowest added this to the 3.0.0 milestone Apr 14, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment