Progress on adding TUMBLE as table value function (CALCITE-3272)

2019-11-14 Thread Rui Wang
Hi community, I have created a new PR ( https://github.com/apache/calcite/pull/1587) to demonstrate the progress of TUMBLE table value function (CALCITE-3272). Julian suggested me to have a working version that adds a stream.iq and have an enumerable implementation. Those are in the PR. High

Re: [DISCUSS] Towards Avatica 1.16.0

2019-11-14 Thread Francis Chuang
Good idea. I think other outdated dependencies (if any) can also be updated in this change as well. On 15/11/2019 3:04 pm, Julian Hyde wrote: Can we also upgrade Jetty? I tried to upgrade Calcite [1] but it gets runtime errors because Avatica is on an earlier version. Julian [1]

Re: [DISCUSS] Towards Avatica 1.16.0

2019-11-14 Thread Julian Hyde
Can we also upgrade Jetty? I tried to upgrade Calcite [1] but it gets runtime errors because Avatica is on an earlier version. Julian [1] https://github.com/apache/calcite/pull/1580 > On Nov 11, 2019, at 12:01 PM, Vladimir Sitnikov > wrote: >

Calcite-Master - Build # 1436 - Failure

2019-11-14 Thread Apache Jenkins Server
The Apache Jenkins build system has built Calcite-Master (build #1436) Status: Failure Check console output at https://builds.apache.org/job/Calcite-Master/1436/ to view the results.

[jira] [Created] (CALCITE-3506) Avatica: exception in the writetoProtoWithType function

2019-11-14 Thread Enrique Saurez (Jira)
Enrique Saurez created CALCITE-3506: --- Summary: Avatica: exception in the writetoProtoWithType function Key: CALCITE-3506 URL: https://issues.apache.org/jira/browse/CALCITE-3506 Project: Calcite

Re: Avatica: exception in the writetoProtoWithType function

2019-11-14 Thread Michael Mior
Unfortunately I'm not too familiar with Avatica, but I would suggest that you file a JIRA issue so discussion can continue there. Thanks! Regards https://issues.apache.org/jira/projects/CALCITE/issues -- Michael Mior mm...@apache.org Le jeu. 14 nov. 2019 à 13:20, Enrique Saurez a écrit : > >

Re: CALCITE-2905: Maven -> Gradle: any thoughts

2019-11-14 Thread Vladimir Sitnikov
The Gradle build scripts are ready ( https://github.com/apache/calcite/pull/1571 ) There might be minor issues like "a resource file missing in one of the jars", however, I expect there will be no more major changes. Unfortunately, there are changes that prevent building with both Maven and

Re: [ANNOUNCE] Haisheng Yuan joins Calcite PMC

2019-11-14 Thread Julian Hyde
Welcome, Haisheng! As Stamatis said, you have been a wise voice in setting the direction of the project, and helpful in discussions. > On Nov 11, 2019, at 9:01 PM, Matt Wang wrote: > > Congratulations! > > > --- > Best, > Matt Wang > > > On 11/12/2019 12:36,Haisheng Yuan wrote: > Thanks,

Re: CsvAdapter (Json content) from String / InputStream

2019-11-14 Thread Andrei Sereda
> I see that this feature request relates to Source.java and Sources.java, which are in org.apache.calcite.util in core. I'm not planning to change Source.java it already exposes Reader / InputStream > If you add some capability, it is fine to add It to the CSV adapter example, but it is much

Re: CsvAdapter (Json content) from String / InputStream

2019-11-14 Thread Julian Hyde
I see that this feature request relates to Source.java and Sources.java, which are in org.apache.calcite.util in core. If you add some capability, it is fine to add It to the CSV adapter example, but it is much more important that that capability exists in the file adapter. Julian > On Nov

Re: CsvAdapter (Json content) from String / InputStream

2019-11-14 Thread Andrei Sereda
I think the change is straightforward (will not add complexity). On Thu, Nov 14, 2019 at 1:24 PM Julian Hyde wrote: > Remember that CsvAdapter is in the “example” module. Keep it simple. > > The file adapter can also parse CSV files. > > Julian > > > > > On Nov 14, 2019, at 9:40 AM, Andrei

Re: CsvAdapter (Json content) from String / InputStream

2019-11-14 Thread Julian Hyde
Remember that CsvAdapter is in the “example” module. Keep it simple. The file adapter can also parse CSV files. Julian > On Nov 14, 2019, at 9:40 AM, Andrei Sereda wrote: > > Hello, > > Source object already exposes Reader / InputStream API. Probably > JsonEnumerator can be changed to use

Avatica: exception in the writetoProtoWithType function

2019-11-14 Thread Enrique Saurez
Hi! I am using Apache Calcite-Avatica version 1.12 (but the relevant code sections are not different from the master branch), and I am getting the following exception on the client side (but the actual error in on the server side): org.apache.calcite.avatica.AvaticaSqlException: Error -1 (0)

Re: CsvAdapter (Json content) from String / InputStream

2019-11-14 Thread Andrei Sereda
Hello, Source object already exposes Reader / InputStream API. Probably JsonEnumerator can be changed to use those methods. Do you mind creating a JIRA ticket ? I'll take a look. Thanks, Andrei. On Thu, Nov 14, 2019 at 7:45 AM Yanna elina wrote: > Hi guys , > I saw in the code that this nice

[jira] [Created] (CALCITE-3505) Infinite matching of FilterProjectTransposeRule causes stackoverflow

2019-11-14 Thread Jin Xing (Jira)
Jin Xing created CALCITE-3505: - Summary: Infinite matching of FilterProjectTransposeRule causes stackoverflow Key: CALCITE-3505 URL: https://issues.apache.org/jira/browse/CALCITE-3505 Project: Calcite

Re: CALCITE-2905: Maven -> Gradle: any thoughts

2019-11-14 Thread Vladimir Sitnikov
>Currently we have two ways of running/skipping a slow test You wanted to say "at least two", did you? There are properties like calcite.test.druid which target a single test class. >Does this make sense considering the ongoing work of moving to Gradle and JUnit5? The current Gradle-JUnit5

Re: CALCITE-2905: Maven -> Gradle: any thoughts

2019-11-14 Thread Stamatis Zampetakis
Hi Vladimir, Thanks for working on this. I have a quick question regarding fast/slow tests. Currently we have two ways of running/skipping a slow test: * using the SlowTests annotation and using JUnit categories; * using the system property calcite.test.slow. I was planning (WIP in [1]) to

CsvAdapter (Json content) from String / InputStream

2019-11-14 Thread Yanna elina
Hi guys , I saw in the code that this nice adapter makes it possible to make SQL queries on the data JSON https://github.com/apache/calcite/tree/ab71c4cae5a5c3c7d979337a2d38ddaf271aa206/example/csv/src/main/java/org/apache/calcite/adapter/csv But it's limited on File / URL. JsonTable constructor

Re: [DISCUSS] Support Sql Hint for Calcite

2019-11-14 Thread Vladimir Ozerov
Hi Danny, Thank you very much for making this happen. Query hints are a very valuable addition to the product. Regards, Vladimir. вс, 10 нояб. 2019 г. в 05:58, Danny Chan : > Hi, fellows, I’m planning to merge the hints PR in the following week, I’m > very appreciated if you have other more

[jira] [Created] (CALCITE-3504) allow empty ARRAY[] literals

2019-11-14 Thread Pressenna (Jira)
Pressenna created CALCITE-3504: -- Summary: allow empty ARRAY[] literals Key: CALCITE-3504 URL: https://issues.apache.org/jira/browse/CALCITE-3504 Project: Calcite Issue Type: Wish Affects

Re: [DISCUSS] Proposal to add API to force rules matching specific rels

2019-11-14 Thread Vladimir Ozerov
Hi Xing, Thanks for your suggestion. Yes, this may help, and if I get your idea right, I already had it in my reproducer: 1) Create the converted physical input: https://github.com/devozerov/calcite-optimizer/blob/master/src/main/java/devozerov/physical/ProjectPhysicalRule.java#L49 2) Use it in

Re: Re: Re: Problem with converters and possibly rule matching

2019-11-14 Thread Vladimir Ozerov
search place* = search space чт, 14 нояб. 2019 г. в 13:10, Vladimir Ozerov : > Hi Haisheng, > > I double-checked the code. My original version returned false for some > cases, but it didn't affect number of rules calls anyway, so I changed it > to always return true. Please note that if I change

Re: Re: Re: Problem with converters and possibly rule matching

2019-11-14 Thread Vladimir Ozerov
Hi Haisheng, I double-checked the code. My original version returned false for some cases, but it didn't affect number of rules calls anyway, so I changed it to always return true. Please note that if I change the code as you suggested, the test started failing, because bottom-up propagation of