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-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
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: 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

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: [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

[jira] [Created] (CALCITE-2591) EnumerableDefaults#mergeJoin should throw error and not return incorrect results when inputs are not ordered

2018-09-23 Thread Enrico Olivelli (JIRA)
Enrico Olivelli created CALCITE-2591: Summary: EnumerableDefaults#mergeJoin should throw error and not return incorrect results when inputs are not ordered Key: CALCITE-2591 URL: https://issues.apache.org

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

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: [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

[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: 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: 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: 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: 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: 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: 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: 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: [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: [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: [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

[jira] [Created] (CALCITE-2406) In Babel parser: allow to use raw string literals for timestamps as in MySQL

2018-07-09 Thread Enrico Olivelli (JIRA)
Enrico Olivelli created CALCITE-2406: Summary: In Babel parser: allow to use raw string literals for timestamps as in MySQL Key: CALCITE-2406 URL: https://issues.apache.org/jira/browse/CALCITE-2406

[jira] [Created] (CALCITE-2405) In Babel parser: allow to use some reserved keyword as identifier

2018-07-09 Thread Enrico Olivelli (JIRA)
Enrico Olivelli created CALCITE-2405: Summary: In Babel parser: allow to use some reserved keyword as identifier Key: CALCITE-2405 URL: https://issues.apache.org/jira/browse/CALCITE-2405 Project

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: 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

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

[jira] [Created] (CALCITE-2395) Support SELECT xxx FROM TABLE FOR UPDATE syntax

2018-07-03 Thread Enrico Olivelli (JIRA)
Enrico Olivelli created CALCITE-2395: Summary: Support SELECT xxx FROM TABLE FOR UPDATE syntax Key: CALCITE-2395 URL: https://issues.apache.org/jira/browse/CALCITE-2395 Project: Calcite

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 &

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: [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: 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

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-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-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

[jira] [Created] (CALCITE-2261) Switch calcilte-core to JDK8

2018-04-17 Thread Enrico Olivelli (JIRA)
Enrico Olivelli created CALCITE-2261: Summary: Switch calcilte-core to JDK8 Key: CALCITE-2261 URL: https://issues.apache.org/jira/browse/CALCITE-2261 Project: Calcite Issue Type

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-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-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: 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-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: 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-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: 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-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: [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: 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

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: Jekyll container configured for Github site generation

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

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

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]],

[jira] [Created] (CALCITE-2100) Ability not to use DriverManager to run only the planner

2017-12-19 Thread Enrico Olivelli (JIRA)
Enrico Olivelli created CALCITE-2100: Summary: Ability not to use DriverManager to run only the planner Key: CALCITE-2100 URL: https://issues.apache.org/jira/browse/CALCITE-2100 Project: Calcite

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

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: 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: [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 >

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: 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: 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: 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: 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

[jira] [Created] (CALCITE-2061) Support for dynamic parameters in offset/fetch (limit)

2017-11-19 Thread Enrico Olivelli (JIRA)
Enrico Olivelli created CALCITE-2061: Summary: Support for dynamic parameters in offset/fetch (limit) Key: CALCITE-2061 URL: https://issues.apache.org/jira/browse/CALCITE-2061 Project: Calcite

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

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-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 >

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: 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
, > "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: 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

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])

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

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

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

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

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

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:

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:

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: 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
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

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: 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: 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: 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

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: 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

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
ateSqlType(SqlTypeName.VARCHAR)).build(); > } > @Override > public Statistic getStatistic() { >return Statistics.of(15D, ImmutableList.of(), > ImmutableList.of(RelCollations.of(0), RelCollations.of(1))); > } > > } >

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: 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

<    1   2