gr4ve commented on issue #1766: [CALCITE-3745] UnitCompiler can not find required class information. URL: https://github.com/apache/calcite/pull/1766#issuecomment-577305920 @vlsi, I have tried to create a configuration of compiler arguments in Frameworks or CalciteConnection, but it is very difficult to achieve under the current architecture. For example: `EnumerableInterpretable.toBindable(...)` is a static method, It has almost no intersection with the external configuration. Also, like JaninoRelMetadataProvider, it has a static instance of itself and used in VolcanoPlanner. As you've seen, calcite also uses the static variable in many other places. Maybe calcite can make a good abstraction in the next big version to resolve this problem. Currently, such changes have minimal impact on current architecture and keep the code clean.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
