Re: Starting with Apache Calcite as SQLPlanner

2017-09-26 Thread Enrico Olivelli
lot of examples of the use of the planner interface. If you're > > looking for a more comprehensive example, Apache Hive uses Calcite's > > optimizer. > > > > The other things you mentioned should all be possible with Calcite and > > possible to add incrementally. &g

Re: Starting with Apache Calcite as SQLPlanner

2017-09-26 Thread Enrico Olivelli
ive to Calcite an SQL query + Current schema + available indexes > > 2) Calcite which creates a access plan using Calcite AST > > 3) Convert Calcite AST to HerdDB access plan AST > > > > I am not looking at something on the client-side I would like to use > > Calcite on the server > > is is possible ? > > > > Thank you in advance > > > > Enrico Olivelli > > eolive...@apache.org > > >

Starting with Apache Calcite as SQLPlanner

2017-09-26 Thread Enrico Olivelli
query + Current schema + available indexes 2) Calcite which creates a access plan using Calcite AST 3) Convert Calcite AST to HerdDB access plan AST I am not looking at something on the client-side I would like to use Calcite on the server is is possible ? Thank you in advance Enrico

Parser error on trivial JDBC update

2017-11-14 Thread Enrico Olivelli
Hi, with a simple UPDATE like: UPDATE mytable set a=? where b=? I get the error below. The "Table" is a ModifiableTable + ScannableTable, with "a" of type INTEGER and "b" of type VARCHAR Any hint ? Thank you Enrico org.apache.calcite.runtime.CalciteContextException: At line 1, column 30:

JIRA Github integration

2017-11-27 Thread Enrico Olivelli
this we be very useful. I think that just an infra ticket can enable it. Cheers Enrico -- -- Enrico Olivelli

Re: Your IDE?

2017-11-26 Thread Enrico Olivelli
..@gmail.com>: > > > What are you using as your IDE for Calcite testing and development? > > > > (I use mostly Sublime Text with VIM bindings, and CLI tools, but > sometimes > > I use IntelliJ IDEA (it is a very heavy system) ) > > > -- -- Enrico Olivelli

Re: [VOTE] Release apache-calcite-1.15.0 (release candidate 0)

2017-11-29 Thread Enrico Olivelli
thor=Julian Hyde <julianh...@gmail.com>" but: was "" Tests in error: OsAdapterTest.testFiles:110 » Runtime exception while executing [select distin... Enrico Olivelli > > Here is my vote: > > +1 (binding) > > Julian >

Re: Release apache-calcite-1.15.0 (release candidate 0)

2017-11-29 Thread Enrico Olivelli
; https://issues.apache.org/jira/browse/CALCITE-2070>; I don’t consider it > a show-stopper. > > Julian > > > > On Nov 29, 2017, at 3:20 AM, Enrico Olivelli <eolive...@apache.org> > wrote: > > > > 2017-11-29 7:10 GMT+01:00 Julian Hyde <jh...@apache.org j

Re: Towards Calcite release 1.15

2017-11-27 Thread Enrico Olivelli
javacc, > >>>>> maven) or change the “preferred” version. If there are other > libraries > >>> we > >>>>> should upgrade, let’s discuss. At this stage I would like to be > >>> ambitious, > >>>>> and upgrade as far as we can. > >>>>> > >>>>> Julian > >>>>> > >>>>> > >>>>>> On Nov 14, 2017, at 7:29 AM, Michael Mior <mm...@uwaterloo.ca> > wrote: > >>>>>> > >>>>>> Sounds good to me. I have nothing in particular I'm hoping to > include. > >>>>>> Looking forward to ditching JDK 7 :) > >>>>>> > >>>>>> -- > >>>>>> Michael Mior > >>>>>> mm...@apache.org > >>>>>> > >>>>>> 2017-11-13 22:46 GMT-05:00 Julian Hyde <jh...@apache.org>: > >>>>>> > >>>>>>> Last release (1.14) was on 2nd October. I think we should aim for a > >>>>>>> release 1.15 in early December (which means starting an RC vote > around > >>>>>>> November 27), and I volunteer to be release manager. > >>>>>>> > >>>>>>> Does this timing work for everyone? > >>>>>>> > >>>>>>> Any particular features that people would like to include? > >>>>>>> > >>>>>>> I would like to announce that this will be the last release that > will > >>>>>>> support JDK 7[1]. > >>>>>>> > >>>>>>> Julian > >>>>>>> > >>>>>>> [1] https://issues.apache.org/jira/browse/CALCITE-2027 < > >>>>>>> https://issues.apache.org/jira/browse/CALCITE-2027> > >>>>> > >>>>> > >>> > >>> > > > > -- -- Enrico Olivelli

Re: Adding UnsafeRow to Calcite

2017-11-23 Thread Enrico Olivelli
> > > https://github.com/apache/spark/blob/master/sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeRow.java > > Kind regards, > > Inigo Mediavilla > -- -- Enrico Olivelli

Re: Columns with name "value"

2017-11-17 Thread Enrico Olivelli
> Either quote it or rename the column. > > Julian > > > > On Nov 16, 2017, at 6:01 AM, Enrico Olivelli <eolive...@gmail.com> > wrote: > > > > Is it possible to make Calcite SqlParser accept queries on column with > name > > "value" without

Re: Parser error on trivial JDBC update

2017-11-17 Thread Enrico Olivelli
I tried to solve the problem this is my proposal https://github.com/apache/calcite/pull/568 I think this partly resolves CALCITE-1708or at least one test case marked with that issue now is working. But I did not drop the @Ignore annotation 2017-11-17 8:59 GMT+01:00 Enrico Olivelli <eol

Columns with name "value"

2017-11-16 Thread Enrico Olivelli
Is it possible to make Calcite SqlParser accept queries on column with name "value" without quoting ? like insert into mytable(name,value values(1,2) I am using Mysql 5.5 conformance and lex below the error Cheers Enrico org.apache.calcite.sql.parser.SqlParseException: Encountered ", value" at

Support for "limit ?"

2017-11-17 Thread Enrico Olivelli
Hi, I see that Calcite does not support dynamic parameters in LIMIT clause. Is it only a simple feature to be implemented or there is a deep reason ? I see that limit offset and fetch are already represented with RexNode in EnumerableLimit If it is feasible can you give me some hint in order to

BindableTableScan is losing aliased named

2017-11-17 Thread Enrico Olivelli
Hi, I have a ProjectableFilterableTable, it seems to me that the BindablaScan RelNode does not keep track of column name aliases in the original projection Example: Query:SELECT k1 theKey FROM tblspace1.tsql where k1 ='mykey2' -- Logical Plan LogicalProject(THEKEY=[$0])

Re: BindableTableScan is losing aliased named

2017-11-17 Thread Enrico Olivelli
, > "THEKEY=5"); } > It passes for me. > You'll need the updated version of this test class, because it was > recently refactored. > It's usually easier for other people to test the problem if you create > runnable test cases to show the problem. Em sexta-feir

Re: Columns with name "value"

2017-11-17 Thread Enrico Olivelli
not something > that we would merge into Calcite. > Oh, I don't like forks. I think I will try to live without this Cheers Enrico > > -- > Michael Mior > mm...@apache.org > > 2017-11-17 3:32 GMT-05:00 Enrico Olivelli <eolive...@gmail.com>: > > > I see, bu

Re: BindableTableScan is losing aliased named

2017-11-17 Thread Enrico Olivelli
ted by the user. Select a as b from table ResultSet rs = ... String col = rs.getString("b"); If I am losing the alias I can't support this case. The test case from Luis is working so there is some trick I am missing Enrico > Julian > > > On Fri, Nov 17, 2017 at 9:33 AM, Enric

Re: BindableTableScan is losing aliased named

2017-11-17 Thread Enrico Olivelli
> an error: > SELECT k1 theKey, k1 theKey2 FROM tblspace1.tsql where k1 ='mykey2' > > Regards, > Luis Fernando > > Em sexta-feira, 17 de novembro de 2017 12:27:49 BRST, Enrico Olivelli < > eolive...@gmail.com> escreveu: > > Hi, > I have a ProjectableFiltera

Re: BindableTableScan is losing aliased named

2017-11-17 Thread Enrico Olivelli
gt; Julian > > > On Fri, Nov 17, 2017 at 10:46 AM, Enrico Olivelli <eolive...@gmail.com> > wrote: > > Il ven 17 nov 2017, 19:19 Julian Hyde <jh...@apache.org> ha scritto: > > > >> > IMHO It would better to keep somethin like an EnumerableProjection >

Another error on using unbounded dynamic parameters

2017-11-18 Thread Enrico Olivelli
I found another query sent to my DBMS which causes troubles to Calcite SELECT ? as foo, k1, n1 FROM mytable The result is the usual: "Illegal use of dynamic parameter" This time it is impossible to infer the type of the parameter. I wonder if it would be possible to implement a configuration

Re: BindableTableScan is losing aliased named

2017-11-18 Thread Enrico Olivelli
I solved my problem just by capturing the original rowtype after sql parser. Thanks Enrico Il ven 17 nov 2017, 22:44 Enrico Olivelli <eolive...@gmail.com> ha scritto: > > > Il ven 17 nov 2017, 22:36 Julian Hyde <jh...@apache.org> ha scritto: > >> Translating to Rel

Re: Volcano planner

2017-11-11 Thread Enrico Olivelli
Enrico Olivelli <eolive...@gmail.com>: > Sorry I cannot make it work for INSERT/DELETE/UPDATE > > > This is the error for a DELETE > Qury: DELETE FROM MYTABLE where id=1 > -- Logical Plan > LogicalTableModify(table=[[x, MYTABLE]], operation=[DELETE], > flattened=[tr

Re: Recycling Planner objects

2017-11-10 Thread Enrico Olivelli
adapters. > > I think we need tutorial, documentation and example code to work directly > > with the planner, because it is a lot harder. > > Regards, > > Luis Fernando > >Em quinta-feira, 9 de novembro de 2017 06:52:26 BRST, Enrico Olivelli > < > > eoliv

Aliases for schemas

2017-11-14 Thread Enrico Olivelli
Hi, (I am playing with the planner, you know) I have several "schema" in my system, like "one", "two" So my root schema is built this way: final SchemaPlus rootSchema = Frameworks.createRootSchema(true); for (String tableSpace : listTableSpaces()) { SchemaPlus schema =

Re: Aliases for schemas

2017-11-14 Thread Enrico Olivelli
set it using > ".defaultSchema( rootSchema.getSubSchema( "one" ))", for example. Em > terça-feira, 14 de novembro de 2017 10:59:33 BRST, Enrico Olivelli < > eolive...@gmail.com> escreveu: > > Hi, > (I am playing with the planner, you know) > I hav

Re: Volcano planner

2017-11-13 Thread Enrico Olivelli
. > > Thank you, > Edmon > > On Mon, Nov 13, 2017 at 10:33 AM, Enrico Olivelli <eolive...@gmail.com> > wrote: > > > Thank you all. > > Once I have understood the basic mechanism I am working very well with > > Calcite planner. > > I will report on

Re: Volcano planner

2017-11-13 Thread Enrico Olivelli
critto: > Enrico, > > The documentation on the planner definitely could be improved. There's > already been an issue created for this which can be followed here: > https://issues.apache.org/jira/browse/CALCITE-2048 > > -- > Michael Mior > mm...@apache.org > > 201

Re: Aliases for schemas

2017-11-14 Thread Enrico Olivelli
ing at the root schema. Returns > > null to only consult the root schema". > > So, if you are using Frameworks.newConfigBuilder(), you can set it using > > ".defaultSchema( rootSchema.getSubSchema( "one" ))", for example. Em > > terça-feira, 14 de

Re: Aliases for schemas

2017-11-14 Thread Enrico Olivelli
It works very well ! Thank you Enrico 2017-11-14 14:57 GMT+01:00 Enrico Olivelli <eolive...@gmail.com>: > ok so I can create the rootSchema, populate with all the schemas and then > give to the config builder the "default schema" > I will give it a try > it

Re: Support for "limit ?"

2017-11-20 Thread Enrico Olivelli
I have a proble on JdbcTest, I can't find how to test dynamic parameters. Can you give some hint ? A part from this a will send my patch soon Enrico 2017-11-17 19:41 GMT+01:00 Enrico Olivelli <eolive...@gmail.com>: > > > Il ven 17 nov 2017, 19:29 Julian Hyde <jh...@apac

Re: Parser error on trivial JDBC update

2017-11-15 Thread Enrico Olivelli
and your query is failing > in sql-to-rel translation, just after validation. Can you log a JIRA case > please? > > Julian > > > > On Nov 14, 2017, at 12:45 PM, Enrico Olivelli <eolive...@gmail.com> > wrote: > > > > Hi, > > with a simple UPDATE like:

Re: Parser error on trivial JDBC update

2017-11-15 Thread Enrico Olivelli
here it is https://issues.apache.org/jira/browse/CALCITE-2054 2017-11-15 14:02 GMT+01:00 Enrico Olivelli <eolive...@gmail.com>: > I will do, > It crashes even if I switch to ProjectableFilterableTable I fall into this, > > For updates without parameters and using Projectabl

CREATE TABLE is not supported ?

2017-11-11 Thread Enrico Olivelli
Hi, I am trying to use Calcite as planner, I get this error using latest master. Is create table not supported ? If not maybe I can try to provide an implementation and a PR if someone could give me some hint about where to start Thanks Enrico org.apache.calcite.sql.parser.SqlParseException:

Re: Volcano planner

2017-11-11 Thread Enrico Olivelli
lanReplacer.visit(RelSubset.java:441) at org.apache.calcite.plan.volcano.RelSubset.buildCheapestPlan(RelSubset.java:291) at org.apache.calcite.plan.volcano.VolcanoPlanner.findBestExp(VolcanoPlanner.java:666) I really appreciate your help Enrico 2017-11-09 9:43 GMT+01:00 Enrico Olivelli <eolive...@gmail.com>: > The example from Luis

Recycling Planner objects

2017-11-09 Thread Enrico Olivelli
I am going to use Calcite Planner inside my DBMS. I would like to understand which objects can be reused and which objects are treated in a dynamic fashion. Planner object, is this cachable ? I see i is not stateless. Is it better to alwayes create a new instance or could it be reclycled ? Table

Re: Volcano planner

2017-11-09 Thread Enrico Olivelli
The example from Luis works like a charm. I have some questions,I will start separate threads Thank you Enrico 2017-11-08 21:51 GMT+01:00 Enrico Olivelli <eolive...@gmail.com>: > Luis thank you, > my case is the second one. I want to use Calcite planner internally on a > dat

Re: Volcano planner

2017-11-08 Thread Enrico Olivelli
Hi, I am playing with the planner but I can't get it work for a very simple query. Th table is MYTABLE(id integer, name varchar)definition is given in code snippet the query is "SELECT * FROM MYTABLE" The error is: org.apache.calcite.plan.RelOptPlanner$CannotPlanException: Node

Re: Volcano planner

2017-11-08 Thread Enrico Olivelli
ateSqlType(SqlTypeName.VARCHAR)).build(); > } > @Override > public Statistic getStatistic() { >return Statistics.of(15D, ImmutableList.of(), > ImmutableList.of(RelCollations.of(0), RelCollations.of(1))); > } > > } >

CachingCalciteSchema thread safe ?

2017-12-05 Thread Enrico Olivelli
Hi, I am getting strange NPEs using Calcite 1.15 (snapshot) like this: java.lang.NullPointerException at org.apache.calcite.prepare.PlannerImpl.rootSchema(PlannerImpl.java:312) at org.apache.calcite.prepare.PlannerImpl.createCatalogReader(PlannerImpl.java:291) at

Re: [VOTE] Release apache-calcite-1.15.0 (release candidate 1)

2017-12-06 Thread Enrico Olivelli
+1 (non binding) Checked source tar ball, notes. - mvn clean install, build OK - All tests pass except from one (OsAdapterTest), see the below, not a blocker, it depends on my SO (Linux Fedora 25) - All tests passes on HerdDB Enrico Tests run: 12, Failures: 0, Errors: 1, Skipped: 2, Time

Re: [DISCUSS] Towards Calcite 1.17.0

2018-07-02 Thread Enrico Olivelli
e this will be enough to do the trick, on Apache BookKeeper we moved completely to Gitbox so in that case it is all simpler I will be happy to help Enrico > Julian > > > > On Jun 30, 2018, at 7:42 AM, Enrico Olivelli > wrote: > > > > Jumping on this train &

Supporting SELECT xxx FROM TABLE FOR UPDATE

2018-07-03 Thread Enrico Olivelli
Hi, My need is to support 'FOR UPDATE' syntax, I am using Calcite SQL Parser and Planner only. I will try to work on this case, it is not blocker for me, I would like to provide an implementation for Calcite 1.18 (not ongoing 1.17) I think Calcite does not deal with explicit locks or such low

Re: [DISCUSS] Towards Calcite 1.17.0

2018-06-30 Thread Enrico Olivelli
gt; in its final stages. There was a lot of activity on avatica-1.12 > > from > > >>>>>> a lot of individuals, and the community is in great shape. > > >>>>>> > > >>>>>> Here are the cases listed by Volodymyr, c

Re: Calcite - JDK 8 and Use of Guava

2018-04-26 Thread Enrico Olivelli
his and figured I'd ask before diving in > > head first. I would start with core and then work out to other modules. > > > > [1] https://issues.apache.org/jira/browse/SOLR-11763 > > [2] https://issues.apache.org/jira/browse/CALCITE-2259 > > > > Kevin Risden > > -- -- Enrico Olivelli

Understanding RexCorrelVariable

2018-01-12 Thread Enrico Olivelli
Hi, I am trying to understand the meaning of RexCorrelVariable With this query I get a Plan like this: Query: UPDATE tblspace1.tsql set n2=n2+n2,n3=n3+n2 WHERE k1=? -- Best Plan EnumerableTableModify(table=[[tblspace1, tsql]], operation=[UPDATE], updateColumnList=[[n2, n3]],

Re: Understanding RexCorrelVariable

2018-01-12 Thread Enrico Olivelli
is always the > current row read from a particular input RelNode. > Thank you Julian, it is clear I will play with joins in order to understand deeply this concept. Enrico > > Julian > > > > On Jan 12, 2018, at 7:49 AM, Enrico Olivelli <eolive...@gmail.com> > wrot

Re: Jekyll container configured for Github site generation

2018-02-14 Thread Enrico Olivelli
te/docker Hope that helps Enrico -- -- Enrico Olivelli

Re: SQL Parsing using Apache Calcite

2018-07-30 Thread Enrico Olivelli
ache Calcite? > This is an application which uses Calcite sql parser and planner https://github.com/diennea/herddb/blob/master/herddb-core/src/main/java/herddb/sql/CalcitePlanner.java Look for 'runPlanner' You will find how to parse an SQL statement Hope that helps Enrico Thanks, > Srikanth. > -- -- Enrico Olivelli

Re: JMH dependency vs licensing

2018-08-10 Thread Enrico Olivelli
I think it is fine to use JMH, you are not "redistributing" it, it is here only to run local benchmarks. We have the same in Apache BookKeeper codebase just my 2 cents Enrico Il giorno ven 10 ago 2018 alle ore 16:56 Michael Mior ha scritto: > Perhaps we should just open up a JIRA case on

Re: Arrow Flight

2018-08-17 Thread Enrico Olivelli
5fb1e6ed630af8df96c15397@%3Cdev.arrow.apache.org%3E > > > > Please join that thread if you are interested. > > > > Julian > > > -- -- Enrico Olivelli

Re: [ANNOUNCE] Apache Calcite 1.17.0 released

2018-07-21 Thread Enrico Olivelli
(including release notes) here: > > http://calcite.apache.org/news/2018/07/20/release-1.17.0/ > > We welcome your help and feedback. For more information on how to > report problems, and to get involved, visit the project website at: > > http://calcite.apache.org/ > > Tha

Re: [ANNOUNCE] Apache Calcite 1.17.0 released

2018-07-21 Thread Enrico Olivelli
a new .md file under site/ and I will edit and > publish it. > > If you’re not much of a coder but would like to contribute to Calcite, > this is a great way to get involved. Submit an article describing a feature > that someone else developed. > > Julian > > > On Jul 21, 2018, at

Re: [VOTE] Release apache-calcite-1.17.0 (release candidate 0)

2018-07-16 Thread Enrico Olivelli
+1 (non binding) Checked checksums, Apache RAT and signatures on source candidate package Run tests on Oracle JDK 8 on Fedora 26 without issues Run unit tests of downstream project HerdDB without issues Great work ! Beside question: I needed to import

Re: Babel and Reserved keywords

2018-09-01 Thread Enrico Olivelli
Julian, This is very interesting to me. I had already tracked a feature request, with a very few reserved words to unblock in Babel https://issues.apache.org/jira/browse/CALCITE-2405 Enrico Il giorno sab 1 set 2018 alle ore 01:19 Julian Hyde ha scritto: > As you may now, we have a ‘Babel’ SQL

Re: Reviews please

2018-07-07 Thread Enrico Olivelli
Sorry, You already separated logical steps very cleanly! The PR would be useful for live comments as well Will take a deep look soon Enrico Il sab 7 lug 2018, 23:02 Enrico Olivelli ha scritto: > It would be useful to create a PR so that github creates the live diff and > we can add co

Re: Reviews please

2018-07-07 Thread Enrico Olivelli
fferent people reviewing each. > > I am especially interested in high-level comments about design and > development practices. > > Julian > -- -- Enrico Olivelli

Re: Reviews please

2018-07-08 Thread Enrico Olivelli
s before the release. There are a lot of changes (thinking about java8 syntax) and bugs are likely to be added. I did a scan thru the change and apparently it looks okay but there are tons of small changes. I really like that Calcite is moving away from Guava Great work! Enrico > > > Julian > -- -- Enrico Olivelli

Re: [VOTE] Release apache-calcite-1.16.0 (release candidate 0)

2018-03-14 Thread Enrico Olivelli
+1 (non binding) Signatures and checksums looks good - Downloaded source tar - Run tests on my laptop (Fedora 26) without errors - Run all tests with HerdDB project (which uses Calcite planner and SQL Parser) without issues Cheers Enrico 2018-03-14 9:57 GMT+01:00 Jesus Camacho Rodriguez

Re: JDK 8 syntax

2018-04-10 Thread Enrico Olivelli
gt; > > > > On Mon, Apr 9, 2018 at 12:54 PM Julian Hyde <jh...@apache.org> 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) > > > espe

Re: JDK 8 syntax

2018-04-10 Thread Enrico Olivelli
e idea. It will be great to have Calcite really built on jdk8 and leverage lambdas Cheers Enrico > > Julian > > > On Apr 10, 2018, at 3:37 AM, Enrico Olivelli <eolive...@gmail.com> > wrote: > > > > This is "the" error (see below). Unfortunately just be

Re: Lineage graph from Queries

2018-04-11 Thread Enrico Olivelli
ables it returns one (or more) access plan ? In this case Calcite is very a good choice. I will be happy to provide some example if this is your case. Enrico > > Thanks & Regards, > B Anil Kumar. > -- -- Enrico Olivelli

Re: JDK 8 syntax

2018-04-14 Thread Enrico Olivelli
> > I’d really appreciate that. > > > > Can you please log a JIRA case, and record your observations/progress > > there. > > > > There’s no particular hurry on this one. > > > > Julian > > > > > On Apr 10, 2018, at 06:32, Enrico Olivelli <e

Re: JDK 8 syntax

2018-04-14 Thread Enrico Olivelli
Il sab 14 apr 2018, 18:20 Enrico Olivelli <eolive...@gmail.com> ha scritto: > > > Il sab 14 apr 2018, 17:39 Vova Vysotskyi <vvo...@gmail.com> ha scritto: > >> Hi all, >> >> I think the reason for this issue is that at the compile stage was assumed >&

Re: JDK 8 syntax

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

Re: JDK 8 syntax

2018-04-09 Thread Enrico Olivelli
ira/browse/CALCITE-2027 < > https://issues.apache.org/jira/browse/CALCITE-2027> > > Julian > > -- -- Enrico Olivelli

Re: JDK 8 syntax

2018-04-17 Thread Enrico Olivelli
Enrico Olivelli <eolive...@gmail.com> ha scritto: > > > Il dom 15 apr 2018, 14:22 Vova Vysotskyi <vvo...@gmail.com> ha scritto: > >> I have reproduced it in Janino only and created the issue: >> https://github.com/janino-compiler/janino/issues/47 > >

Re: JDK 8 syntax

2018-04-17 Thread Enrico Olivelli
gt; > Kind regards, > Volodymyr Vysotskyi > > 2018-04-17 15:10 GMT+03:00 Enrico Olivelli <eolive...@gmail.com>: > > > I have tried to add an 'unwrap' method to Schema but then Janino keeps > > breaking for other similar reasons about method overriding with narrower

Re: JDK 8 syntax

2018-04-21 Thread Enrico Olivelli
Patch merged! Welcome to java 8 ! Enrico Il mar 17 apr 2018, 17:09 Enrico Olivelli <eolive...@gmail.com> ha scritto: > Issue > https://issues.apache.org/jira/browse/CALCITE-2261 > > Patch > https://github.com/apache/calcite/pull/667 > > Cheers > Enrico > >

Re: JDK 8 syntax

2018-04-15 Thread Enrico Olivelli
, the > >> project maintainer, has been very good to us over the years. > >> > >> Julian > >> > >> > >> On Sat, Apr 14, 2018 at 9:28 AM, Enrico Olivelli <eolive...@gmail.com> > >> wrote: > >> > Il sab 14 apr 2018, 18:

Re: [ANNOUNCE] Francis Chuang joins Calcite PMC

2018-04-23 Thread Enrico Olivelli
Congrats Francis ! Enrico 2018-04-23 17:25 GMT+02:00 Shuyi Chen : > Congratulations! > > On Mon, Apr 23, 2018 at 7:25 AM Michael Mior wrote: > > > I'm please to announce that Francis has accepted an invitation to join > the > > Calcite PMC. Francis has

Re: JDK 8 syntax

2018-04-17 Thread Enrico Olivelli
Issue https://issues.apache.org/jira/browse/CALCITE-2261 Patch https://github.com/apache/calcite/pull/667 Cheers Enrico 2018-04-17 16:51 GMT+02:00 Enrico Olivelli <eolive...@gmail.com>: > Vova, > I tried to add some "default" methods and all tests are passing (m

Running tests on CI on PR submission

2018-03-04 Thread Enrico Olivelli
Hi, It would be great to run tests on pull requests. It is very simple to setup and it will ease reviews. I know quite well ASF CI (I have write access) and I can help Enrico -- -- Enrico Olivelli

Re: Running tests on CI on PR submission

2018-03-05 Thread Enrico Olivelli
Sorry, did not see Travis running the full suite of tests rollback Enrico 2018-03-05 8:36 GMT+01:00 Enrico Olivelli <eolive...@gmail.com>: > Hi, > It would be great to run tests on pull requests. > It is very simple to setup and it will ease reviews. > > I know quite well

Re: [DISCUSS] Towards Calcite 1.17.0

2018-06-28 Thread Enrico Olivelli
PR is > > created > > CALCITE-2301: Remove the 10-second-timeout restriction in > > ResultSetEnumerable - needs additional rework. > > > > Please let me know, if there are any Jiras should be included into this > > release. > > > > [1] > > > > > https://issues.apache.org/jira/issues/?jql=project%20%3D%20CALCITE%20AND%20status%20%3D%20Resolved%20AND%20resolution%20%3D%20Fixed%20AND%20fixVersion%20%3D%201.17.0 > > [2] https://issues.apache.org/jira/browse/CALCITE-2337 > > > > Kind regards, > > Volodymyr Vysotskyi > > > -- -- Enrico Olivelli

Re: Babel and Reserved keywords

2018-09-29 Thread Enrico Olivelli
I would add 'second' as non reserved word. Select foo as first, bar as second from mytable ... Thank you Enrico Il giorno lun 3 set 2018 alle ore 01:28 Michael Mior ha scritto: > > I wrote a simple script that just goes through all the keywords one by one > and tries them (by running the

Re: Help with EnumerableMergeJoinRule which is losing a RelCollection trait

2018-09-29 Thread Enrico Olivelli
y much ! Enrico > > > Στις Παρ, 28 Σεπ 2018 στις 11:39 μ.μ., ο/η Enrico Olivelli < > eolive...@gmail.com> έγραψε: > > > Il ven 28 set 2018, 23:00 Stamatis Zampetakis ha > > scritto: > > > > > Hi Enrico, > > > > > > I

Re: Help with EnumerableMergeJoinRule which is losing a RelCollection trait

2018-09-28 Thread Enrico Olivelli
passing only EnumerableConvention as desiredTraits. I can't find the 'Trait' for RelCollation, there is no constant like EnumerableConvention. Is there anya special way to instantiate it? I can't find and examplethe in Calcite tests Thank you so much Enrico > Best, > Stamatis > > Στις Παρ, 28

Re: Help with EnumerableMergeJoinRule which is losing a RelCollection trait

2018-09-24 Thread Enrico Olivelli
re using a somewhat creative way of planning the whole thing as > Enumerable then converting the plan relations to your own format. > Uh Othewise I should have created my own Convention, but as far I understand it is not really needed, in the future maybe we refactor the whole integration with Calcite Enrico > Vladimir > -- -- Enrico Olivelli

Help with EnumerableMergeJoinRule which is losing a RelCollection trait

2018-09-22 Thread Enrico Olivelli
Hi, We found a strange behaviour in an execution plan, basially we have an EnumerableMergeJoin which has as input two non-sorted EnumerableTableScan all the details are in this issue on HerdDB https://github.com/diennea/herddb/issues/262#issuecomment-423590573 Cut and paste from the issue in

Re: CALCITE-2458 Use of Kotlin for unit tests

2018-09-19 Thread Enrico Olivelli
co > Julian>Please withdraw the remark and apologize. Your point has merit, but > there is a way to make it without making it personal. > > I'm afraid I can't understand what you say here. Would you please > elaborate? > > Vladimir > -- -- Enrico Olivelli

Re: [DISCUSS] Problem with 'assert' ignored in production, leading to inconsistent query results

2018-09-21 Thread Enrico Olivelli
ritical code. > > > > We definitely can afford checks that take a couple of CPU cycles to > perform. > > > > Vladimir > -- -- Enrico Olivelli

Re: Help with EnumerableMergeJoinRule which is losing a RelCollection trait

2018-09-23 Thread Enrico Olivelli
the RelCollationTraitDef. > I will try. Thank you Enrico > > Best, > Stamatis > > Στις Κυρ, 23 Σεπ 2018 στις 8:24 π.μ., ο/η Enrico Olivelli < > eolive...@gmail.com> έγραψε: > > > Il giorno dom 23 set 2018 alle ore 00:11 Stamatis Zampetakis > > ha scritto:

Re: Help with EnumerableMergeJoinRule which is losing a RelCollection trait

2018-09-23 Thread Enrico Olivelli
but just in case are there any kind of > statistics on these tables. > I've seen a similar case where the statistics of the table were declaring > that some columns were sorted without this being true. > I just want to make sure that this is not the case here. > > Best, > S

Re: [DISCUSS] Problem with 'assert' ignored in production, leading to inconsistent query results

2018-09-23 Thread Enrico Olivelli
Il giorno ven 21 set 2018 alle ore 20:03 Enrico Olivelli ha scritto: > > > > Il ven 21 set 2018, 19:45 Julian Hyde ha scritto: >> >> Well, maybe we can. >> >> But look at the premise of the question: "Problem with 'assert' >> ignored in pr

[DISCUSS] Problem with 'assert' ignored in production, leading to inconsistent query results

2018-09-21 Thread Enrico Olivelli
Hi, one of my customer got into a very weird situation when a merge join was returning less results than expected. Reproducing the issue on a unit test the query execution resulted in an assertion error so that is was easy to find the problem (a bug in the top level application in this case).

Re: Help with EnumerableMergeJoinRule which is losing a RelCollection trait

2018-09-24 Thread Enrico Olivelli
Il lun 24 set 2018, 13:50 Enrico Olivelli ha scritto: > > > Il dom 23 set 2018, 22:22 Vladimir Sitnikov > ha scritto: > >> Enrico, >> >> 1) EnumerableMergeJoinRule does not seem to work (see >> https://issues.apache.org/jira/browse/CALCITE-2592 ). >&

Re: [ANNOUNCE] New Calcite PMC chair: Francis Chuang

2018-12-20 Thread Enrico Olivelli
t; >> resigned, and Francis was duly elected by the PMC and approved > unanimously > >> by the Board. > >> > >> Please join me in congratulating Francis! > >> > >> -Michael > >> > >> PS - This should not be indicative of a

Re: [VOTE] Release apache-calcite-1.18.0 (release candidate 0)

2018-12-04 Thread Enrico Olivelli
+1 (non binding) - compiled from the staged sources and run tests on linux, all passed - run tests of downstream project (HerdDB) - checked signatures and checksums great work ! Enrico Il giorno mar 4 dic 2018 alle ore 10:00 Julian Hyde ha scritto: > > Hi all, > > I have created a build for

Re: [VOTE] Release apache-calcite-1.18.0 (release candidate 1)

2018-12-07 Thread Enrico Olivelli
+1 (non binding) - checked signature and checksum - built from source, all tests pass on OracleJDK8 on Linux - all tests from downstream project (HerdDB) are passing Still great work ! Thank you Julian for driving the release Enrico Il giorno ven 7 dic 2018 alle ore 02:44 Francis Chuang ha

Re: calcite-avatica git commit: [CALCITE-2412] Add appveyor.yml to have tests on Windows against jdk1.8, jdk9, jdk10 Add Appveyor badge Add -DskipDockerCheck because of CALCITE-2385 and to make it syn

2018-11-19 Thread Enrico Olivelli
>>> > >>> On 20/11/2018 8:00 am, Michael Mior wrote: > >>>> Sorry, that link should have been > >>>> https://github.com/phillipuniverse/githook-maven-plugin. Anyway, I don't > >>>> have experience with any particular plugin,

Re: calcite-avatica git commit: [CALCITE-2412] Add appveyor.yml to have tests on Windows against jdk1.8, jdk9, jdk10 Add Appveyor badge Add -DskipDockerCheck because of CALCITE-2385 and to make it syn

2018-11-19 Thread Enrico Olivelli
Il giorno lun 19 nov 2018 alle ore 22:41 Enrico Olivelli ha scritto: > > We can run the script in Travis, so that we will refuse any commit > message not compliant with the "specs" > In Travis you can run a shell script with uses directly the 'git' command What about this

Re: calcite-avatica git commit: [CALCITE-2412] Add appveyor.yml to have tests on Windows against jdk1.8, jdk9, jdk10 Add Appveyor badge Add -DskipDockerCheck because of CALCITE-2385 and to make it syn

2018-11-19 Thread Enrico Olivelli
; > > API, so ResultSetEnumerable.setDynamicParam could be removed > altogether. > > > > I agree. The commit didn’t seem quite perfect to me either. However, it > > seemed to be progress. Log an Avatica JIRA if you have ideas for how to > > improve it further. Since it will be in Avatica it will take a while to > > bubble through the release cycle. > > > > Julian > > > > > -- -- Enrico Olivelli

Re: Avoid re-generate Parser.jj on each build if the parser was not modified

2018-09-15 Thread Enrico Olivelli
nd I couldn't find a continuation. > > Vladimir > -- -- Enrico Olivelli

Re: CALCITE-2458 Use of Kotlin for unit tests

2018-09-18 Thread Enrico Olivelli
te are organised... I sponsor the idea of having domain specific test languages like QUIDEM which make it simpler to write complex assertions in this SQL/Calcite environment. So from me it is more like a -0 for Kotlin now My two cents Enrico > Vladimir > -- -- Enrico Olivelli

Re: [VOTE] Release apache-calcite-1.18.0 (release candidate 2)

2018-12-19 Thread Enrico Olivelli
+1 (non binding) built from source artifacts, run tests, all passed on Linux fedora 28 run tests of HerdDB, all is ok checked signatures and checksums, all is ok great work Enrico Il giorno mer 19 dic 2018 alle ore 13:32 Stamatis Zampetakis ha scritto: > > +1 (non binding) > > System: Ubuntu

Re: [VOTE] Release apache-calcite-1.19.0 (release candidate 2)

2019-03-21 Thread Enrico Olivelli
great work Kevin +1 (non binding) checked signatures/checksum built from source on JDK11 (AdoptOpenJDK 11.0.2) on Linux, all tests passed. all tests of downstream project HerdDB passed, 100% compatible, upgrading from 1.17.0 Regards Enrico Il giorno mer 20 mar 2019 alle ore 22:58 Francis

Re: new RuntimeException(Throwable) vs SneakyThrows

2019-02-13 Thread Enrico Olivelli
Vladimir, Il giorno mer 13 feb 2019 alle ore 21:49 Vladimir Sitnikov ha scritto: > > Hi, > > I've recently discovered that new RuntimeException(Throwable) results in a > duplicated error messages. > In fact Throwable#(Throwable) just calls cause.getMessage() and uses > it as its own message. > >

Re: Calcite Plan Caching

2019-02-07 Thread Enrico Olivelli
Paul, Il giorno mer 6 feb 2019, 23:02 Paul Trepagnier ha scritto: > I am using calcite right now to test some data federation scenarios. I > have several collections of Java objects that I am querying right now. It > works beautifully, but it seems like Calcite does a ton of work with each >

Re: [ANNOUNCE] New committer: Stamatis Zampetakis

2019-01-30 Thread Enrico Olivelli
Kudos ! Enrico Il giorno mer 30 gen 2019, 19:27 Julian Hyde ha scritto: > Welcome, Stamatis! In addition to your code contributions, I have been > appreciating your wise & moderating contributions to technical discussions, > and answers to people’s questions on the dev list. > > Julian > > > >

  1   2   >