birschick-bq opened a new pull request #2449:
URL: https://github.com/apache/calcite/pull/2449


   ## Description
   
   This change addresses the issue of the Janino library not being able to load 
the correct class loader.
   
   - Passes the class loader of the calling class.
   - Updates the Janino version to 3.1.4
   
   ## Notes
   
   - There is a [know 
issue](https://github.com/janino-compiler/janino/pull/150) in the Janino 
library where the class loader is not being properly passed to underlying 
methods. A [fix ](https://github.com/janino-compiler/janino/pull/150) has been 
applied but not release yet.
   - As of version 3.1.3, a number of Calcite test fail with 
`org.codehaus.commons.compiler.InternalCompilerException`. This pull request is 
here to help highlight the issue and work with Janino team to resolve that 
issue.
   - So this pull request will need to be updated with a newer version of 
Janino where both issue are resolved.
   
   ## Related Issue
   
   - https://issues.apache.org/jira/browse/CALCITE-3745
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to