[jira] [Created] (CALCITE-4947) Checkstyle fails on classes generated by Intellij when using option "build and run [tests] using Intellij IDEA"

2021-12-17 Thread Thomas Rebele (Jira)
Thomas Rebele created CALCITE-4947: -- Summary: Checkstyle fails on classes generated by Intellij when using option "build and run [tests] using Intellij IDEA" Key: CALCITE-4947 URL:

RE: Re: [jira] [Created] (CALCITE-4898) Upgrading Elasticsearch version from 7.0.1 to 7.15 in Elasticsearch Adapter

2021-12-17 Thread 胡哲
Hi, Julian. Your concern sounds reasonable, so I run some tests about those dependencies. Adding "org.elasticsearch.plugin:elasticsearch-scripting-painless-spi” and removing “org.codelibs.elasticsearch.module:scripting-painless-spi”, the test results are OK. However,

[jira] [Created] (CALCITE-4946) Add method RelBuilder.size()

2021-12-17 Thread Julian Hyde (Jira)
Julian Hyde created CALCITE-4946: Summary: Add method RelBuilder.size() Key: CALCITE-4946 URL: https://issues.apache.org/jira/browse/CALCITE-4946 Project: Calcite Issue Type: Bug

Re: Creating a parser for Big query

2021-12-17 Thread Maxim Gramin
Can you show the query? I don't see this one. On Thu, Dec 16, 2021 at 10:12 PM Arash Bizhan zadeh wrote: > Hi, > I am trying to create a parser to parse big query statements like this - > this is kotlin btw: > > val sqlParserConfig = BigQuerySqlDialect( > >

Re: Creating a parser for Big query

2021-12-17 Thread Arash Bizhan zadeh
This is my test query SELECT IF(CAST(age as numeric) < 0.5, name, last_name) as customer_name FROM 307320.test_dataset.customers On Fri, Dec 17, 2021 at 5:47 AM Maxim Gramin wrote: > Can you show the query? I don't see this one. > > On Thu, Dec 16, 2021 at 10:12 PM Arash Bizhan zadeh >

Re: Failing task :linq4j:jandexMain when executing CheckerFramework

2021-12-17 Thread Thomas Rebele
Hello, thank you, Vladimir Sitnikov, for the hint with the javaccGeneratedPatterns in build.gradle.kts. I discovered the cause for the generated files. I forgot that I had activated the option "build and run [tests] using Intellij IDEA". Exclusion rules to be added by CALCITE-4947

Re: Creating a parser for Big query

2021-12-17 Thread Julian Hyde
Or use Babel parser. > On Dec 17, 2021, at 9:35 AM, Maxim Gramin wrote: > > It looks like the parser needs to be extended > https://calcite.apache.org/docs/adapter.html#extending-the-parser > > On Fri, Dec 17, 2021 at 5:34 PM Arash Bizhan zadeh > wrote: > >> This is my test query >> >>

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-17 Thread Julian Hyde
Rui, Even though https://issues.apache.org/jira/browse/CALCITE-4898 is marked fixed, there’s some fallout regarding maven repositories and category X. That needs to be sorted out before RC0. See [1]. I’m working on getting

[jira] [Created] (CALCITE-4948) Revert Elasticsearch to 7.10.2 to avoid extra repo and category x license

2021-12-17 Thread Jacques Nadeau (Jira)
Jacques Nadeau created CALCITE-4948: --- Summary: Revert Elasticsearch to 7.10.2 to avoid extra repo and category x license Key: CALCITE-4948 URL: https://issues.apache.org/jira/browse/CALCITE-4948

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-17 Thread Ruben Q L
Hello, FYI https://issues.apache.org/jira/browse/CALCITE-4737 has been merged. Regards, Ruben On Fri, Dec 17, 2021 at 7:40 PM Julian Hyde wrote: > Rui, > > Even though https://issues.apache.org/jira/browse/CALCITE-4898 < > https://issues.apache.org/jira/browse/CALCITE-4898> is marked fixed,

Re: Creating a parser for Big query

2021-12-17 Thread Maxim Gramin
It looks like the parser needs to be extended https://calcite.apache.org/docs/adapter.html#extending-the-parser On Fri, Dec 17, 2021 at 5:34 PM Arash Bizhan zadeh wrote: > This is my test query > > SELECT IF(CAST(age as numeric) < 0.5, name, last_name) as customer_name > FROM

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-17 Thread Jacques Nadeau
I've merged CALCITE-4948 to mitigate the concerns around CALCITE-4898 and maven repositories and category x licenses. On Fri, Dec 17, 2021 at 1:02 PM Rui Wang wrote: > For the list of 1.29 fix Jira, several Jira have been marked as resolved > since this Monday. I will do another pass today but

[jira] [Created] (CALCITE-4949) Evaluate if org.codelibs.elasticsearch.module:lang-painless is an appropriate dependency

2021-12-17 Thread Jacques Nadeau (Jira)
Jacques Nadeau created CALCITE-4949: --- Summary: Evaluate if org.codelibs.elasticsearch.module:lang-painless is an appropriate dependency Key: CALCITE-4949 URL: https://issues.apache.org/jira/browse/CALCITE-4949

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-17 Thread Francesco Gini
https://issues.apache.org/jira/browse/CALCITE-4907 is in review. I believe I have addressed all the comments. Let me know if I missed anything. On Fri, 17 Dec 2021 at 22:42, Jacques Nadeau wrote: > I've merged CALCITE-4948 to mitigate the concerns around CALCITE-4898 and > maven repositories

Re: [DISCUSS] State of the project 2021

2021-12-17 Thread Forward Xu
Thank you very much Ruben for his work. Ruben PMC chair +1 Forwardxu Haisheng Yuan 于2021年12月11日周六 09:03写道: > Hi Calcite community members, > > It has been 6 years since Calcite graduated to a top level Apache project. > I am so excited to witness how vivid the community has become and how

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-17 Thread Rui Wang
For the list of 1.29 fix Jira, several Jira have been marked as resolved since this Monday. I will do another pass today but probably not all those will make the 1.29.0 release. -Rui On Fri, Dec 17, 2021 at 11:57 AM Ruben Q L wrote: > Hello, > > FYI

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-17 Thread Zhe Hu
Hi, Rui. CALCITE-4860 will be fixed in next release, you can pass it currently. Thanks for your work! Best, ZheHu On 12/18/2021 06:08,Francesco Gini wrote: https://issues.apache.org/jira/browse/CALCITE-4907 is in review. I believe I have addressed all the comments. Let me know if I missed

Re: Creating a parser for Big query

2021-12-17 Thread Arash Bizhan zadeh
In the documentation section - https://calcite.apache.org/docs/reference.html - in "Dialect Specific Operator" I can see that "IF" is listed for Big-Query. Based on this I assumed that it should be supported out of the box without extentind the parser. It says that I need to specify a "fun"

Re: Creating a parser for Big query

2021-12-17 Thread Arash Bizhan zadeh
I thought I was already using Babel parser - .withParserFactory(SqlBabelParserImpl.FACTORY) . Care to give me a code snippet? On Fri, Dec 17, 2021 at 2:45 PM Julian Hyde wrote: > Or use Babel parser. > > > On Dec 17, 2021, at 9:35 AM, Maxim Gramin > wrote: > > > > It looks like the parser

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-17 Thread Rui Wang
Thanks Julian! Now we will just decide whether to merge CALCITE-4907 or not, then I will lock the main branch and start the release process. -Rui On Fri, Dec 17, 2021 at 10:29 PM Julian Hyde wrote: > Rui, I have merged CALCITE-4946. > > > On Dec 17, 2021, at 10:21 PM, Rui Wang wrote: > > >

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-17 Thread Vladimir Ozerov
Hi, Could you please also merge https://github.com/apache/calcite/pull/2464 ? I guess it got stuck somehow, as all the comments are addressed. This issue forced us to create a copy of the AggregateUnionTransposeRule in our project. Regards, Vladimir Сб, 18 дек. 2021 г. в 09:51, Rui Wang : >

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-17 Thread Rui Wang
Thanks Zhe. Hi Francesco, I left a comment in your PR. Can you take a look? -Rui On Fri, Dec 17, 2021 at 3:32 PM Zhe Hu wrote: > Hi, Rui. > CALCITE-4860 will be fixed in next release, you can pass it currently. > Thanks for your work! > > > Best, > ZheHu > > > > > > > On 12/18/2021

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-17 Thread Julian Hyde
Rui, I have merged CALCITE-4946. > On Dec 17, 2021, at 10:21 PM, Rui Wang wrote: > > Thanks Zhe. > > Hi Francesco, > > I left a comment in your PR. Can you take a look? > > -Rui > > On Fri, Dec 17, 2021 at 3:32 PM Zhe Hu wrote: > >> Hi, Rui. >> CALCITE-4860 will be fixed in next release,

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-17 Thread xiong duan
Hi, Vladimir. The PR[https://github.com/apache/calcite/pull/2464] looks good. If you have free time Please resolve the conflicts. Then we can merge it. Vladimir Ozerov 于2021年12月18日周六 15:20写道: > Hi, > > Could you please also merge > https://github.com/apache/calcite/pull/2464 ? I guess it got