Re: JDK 8 syntax

2018-08-28 Thread Vova Vysotskyi
gt;>> visitParenthesizedExpr > >>>>>>>>>> ession( > >>>>>>>>>>>>>>>> UnitCompiler.java:4074) > >>>>>>>>>>>>>>>>>> at &

Re: JDK 8 syntax

2018-08-28 Thread Julian Hyde
gt;>>>>>> accept(Java.java:4753) >>>>>>>>>>>>>>>>>> at >>>>>>>>>>>>>>>>> org.codehaus.janino.UnitCompiler$12.visitLvalue( >>>>>>>>>>>>>> UnitCompiler.java:4

Re: JDK 8 syntax

2018-08-28 Thread Vova Vysotskyi
n.accept(Java.java:4902) > >>> > >> >> >>> > >>>> at org.codehaus.janino.UnitCompiler.compileGet( > >>> > >> >> >>> > >> UnitCompiler.java:4070) > >>> > >> >> >>> >

Re: JDK 8 syntax

2018-04-21 Thread Enrico Olivelli
; >>> org.codehaus.janino.UnitCompiler.compileGet2(UnitCompiler. >>> > >> >> java:4696) >>> > >> >> >>> > >>>> at org.codehaus.janino.UnitCompiler.access$7800( >>> >

Re: JDK 8 syntax

2018-04-17 Thread Enrico Olivelli
>>> at org.codehaus.janino.UnitCompiler.compileGet( >> > >> >> >>> > >> UnitCompiler.java:4070) >> > >> >> >>> > >>>> at >> > >> >> >>> > >>> org.codehaus.janino

Re: JDK 8 syntax

2018-04-17 Thread Enrico Olivelli
gt;> ession( > > >> >> >>> > >> UnitCompiler.java:4083) > > >> >> >>> > >>>> at > > >> >> >>> > >>> org.codehaus.janino.UnitCompiler$12$1. > > visitParenthesizedExpr > > >> >> ession( > > >>

Re: JDK 8 syntax

2018-04-17 Thread Vova Vysotskyi
; >>> > >>> org.codehaus.janino.UnitCompiler$12.visitLvalue( > >> >> >>> > UnitCompiler.java:4074) > >> >> >>> > >>>> at > >> >> >>> > >>> org.codeha

Re: JDK 8 syntax

2018-04-17 Thread Enrico Olivelli
>> >> >>> > UnitCompiler.java:4070) >> >> >>> > >>>> at org.codehaus.janino.Java$Lvalue.accept(Java.java:3977) >> >> >>> > >>>> at org.codehaus.janino.UnitCompiler.compileGet( >> >> >>>

Re: JDK 8 syntax

2018-04-15 Thread Enrico Olivelli
> at org.codehaus.janino.UnitCompiler.access$6100( > >> >>> > >> UnitCompiler.java:212) > >> >>> > >>>> at > >> >>> > >>> org.codehaus.janino.UnitCompiler$11$1.visitFieldAccess( > >> UnitComp

Re: JDK 8 syntax

2018-04-15 Thread Vova Vysotskyi
dehaus.janino.UnitCompiler$11.visitLvalue( >> >>> > UnitCompiler.java:3942) >> >>> > >>>> at >> >>> > >>> org.codehaus.janino.UnitCompiler$11.visitLvalue( >> >>> > UnitCompiler.java:3938) >> >>>

Re: JDK 8 syntax

2018-04-14 Thread Vova Vysotskyi
t;> org.codehaus.janino.UnitCompiler.compileContext2( > >>> > UnitCompiler.java:4047) > >>> > >>>> at org.codehaus.janino.UnitCompiler.access$6200( > >>> > >> UnitCompiler.java:212) > >>>

Re: JDK 8 syntax

2018-04-14 Thread Julian Hyde
.visitFieldAccessExpression( >>> > >> UnitCompiler.java:3942) >>> > >>>> at >>> > >>> >>> org.codehaus.janino.Java$FieldAccessExpression.accept(Java.java:4468) >>> > >>>> at >>> >

Re: JDK 8 syntax

2018-04-14 Thread Enrico Olivelli
ler.java:5252) >> > >>>> at >> > >>> org.codehaus.janino.UnitCompiler.compileGet2(UnitCompiler.java:4842) >> > >>>> at org.codehaus.janino.UnitCompiler.access$8300( >> > >> UnitCompiler.java:212) >> &

Re: JDK 8 syntax

2018-04-14 Thread Enrico Olivelli
ompileGet( > > >> UnitCompiler.java:4070) > > >>>> at > > >>> org.codehaus.janino.UnitCompiler.compileGetValue( > > UnitCompiler.java:5253) > > >>>> at org.codehaus.janino.UnitCompiler.compile2( > > UnitCompiler.jav

Re: JDK 8 syntax

2018-04-14 Thread Vova Vysotskyi
> >>> org.codehaus.janino.Java$LocalVariableDeclarationStatem > >> ent.accept(Java.java:3351) > >>>> at org.codehaus.janino.UnitCompiler.compile(UnitCompiler.java:1466) > >>>> at org.cod (stack truncated) > >>> >

Re: JDK 8 syntax

2018-04-10 Thread Julian Hyde
<cademarkeg...@gmail.com> ha >>>>> scritto: >>>>> >>>>>> Not sure if this is related, but it seems like Janino doesn't support >>>>> Java8 >>>>>> lambdas under "Limitations" in http://janino-compiler.github. &

Re: JDK 8 syntax

2018-04-10 Thread Enrico Olivelli
er). I > >> have > >>> not much time these days. > >>> > >>> Anyhow I am testing my apps on jdk10 and Calcite is working like a > charm > >> ! > >>> > >>> Enrico > >>> > >>>> > >>>

Re: JDK 8 syntax

2018-04-10 Thread Enrico Olivelli
> > > > > > >> 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 perh

Re: JDK 8 syntax

2018-04-09 Thread Cade Markegard
ange 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 > > > > > > > >

Re: JDK 8 syntax

2018-04-09 Thread Cade Markegard
ore/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, 20

Re: JDK 8 syntax

2018-04-09 Thread Julian Hyde
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 <eolive...@gmail.com> >> wrote: &g

Re: JDK 8 syntax

2018-04-09 Thread Enrico Olivelli
<eolive...@gmail.com> > wrote: > > > > Il lun 9 apr 2018, 19:05 Julian Hyde <jh...@apache.org> ha scritto: > > > >> Has anyone had any luck using JDK 8 syntax (e.g. “->” for lambdas) in > >> Calcite? > >> > >> In core/pom.

Re: JDK 8 syntax

2018-04-09 Thread Julian Hyde
test”. Julian > On Apr 9, 2018, at 12:15 PM, Enrico Olivelli <eolive...@gmail.com> wrote: > > Il lun 9 apr 2018, 19:05 Julian Hyde <jh...@apache.org> ha scritto: > >> Has anyone had any luck using JDK 8 syntax (e.g. “->” for lambdas) in >> Calcite? >

Re: JDK 8 syntax

2018-04-09 Thread Enrico Olivelli
Il lun 9 apr 2018, 19:05 Julian Hyde <jh...@apache.org> 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 bre

JDK 8 syntax

2018-04-09 Thread Julian Hyde
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. Related: https://issues.apache.org/jira/browse/CALCITE-2