Not sure if this is related, but it seems like Janino doesn't support Java8 lambdas under "Limitations" in http://janino-compiler.github.io/janino/
On Mon, Apr 9, 2018 at 12:54 PM Julian Hyde <[email protected]> wrote: > I didn’t try. We’re on janino 2.7.6. The 2.x line only goes up to 2.7.8. > I’m not eager to try upgrading to 3.x (latest release is 3.0.8) especially > as they don’t explicitly mention fixing a JDK 8 issue. > > http://janino-compiler.github.io/janino/changelog.html < > http://janino-compiler.github.io/janino/changelog.html> > > > > On Apr 9, 2018, at 12:42 PM, Enrico Olivelli <[email protected]> > wrote: > > > > Il lun 9 apr 2018, 21:37 Julian Hyde <[email protected] <mailto: > [email protected]>> ha scritto: > > > >> Don’t have the error stack, but janino could not compile generated code > >> because it said an “unwrap” method did not exist. > >> > >> There were no lambdas. I think perhaps that version of janino could not > >> fully understand the Calcite class files. > >> > >> To reproduce, change 2 lines in core/pom.xml and run “mvn test”. > >> > > > > Did you try to update to latest Janino? > > It seems to that we have an old version (2.7.6) > > > > Enrico > > > >> > >> Julian > >> > >> > >>> On Apr 9, 2018, at 12:15 PM, Enrico Olivelli <[email protected]> > >> wrote: > >>> > >>> Il lun 9 apr 2018, 19:05 Julian Hyde <[email protected]> ha scritto: > >>> > >>>> Has anyone had any luck using JDK 8 syntax (e.g. “->” for lambdas) in > >>>> Calcite? > >>>> > >>>> In core/pom.xml I changed source=1.7 target=1.7 to source=8 target=8 > and > >>>> it allowed the syntax but it seemed to break any tests that relied on > >>>> janino. > >>>> > >>> > >>> Julian, > >>> Could you please share a stacktrace of your error? > >>> As far as I know using lambdas is not only a matter of syntactic sugar > >> but > >>> they are more efficient than simple anonymous classes. > >>> > >>> Cheers > >>> Enrico > >>> > >>> > >>>> Related: https://issues.apache.org/jira/browse/CALCITE-2027 < > >>>> https://issues.apache.org/jira/browse/CALCITE-2027> > >>>> > >>>> Julian > >>>> > >>>> -- > >>> > >>> > >>> -- Enrico Olivelli > >> > >> -- > > > > > > -- Enrico Olivelli > >
