Re: Errored: apache/calcite#12834 (main - 94dc303)

2022-06-23 Thread Francis Chuang
Looking at the jobs on GH actions, I don't think we are testing all the OpenJDK versions that are being tested on Travis: https://github.com/apache/calcite/actions/runs/2551147939 I think over there's some drift between the Travis and GH Actions config and they are not exactly equivalent /

Re: Errored: apache/calcite#12834 (main - 94dc303)

2022-06-23 Thread Julian Hyde
This is a problem on the Travis side, right? I can’t see how this one-line change could have broken anything. > On Jun 23, 2022, at 4:10 PM, Francis Chuang wrote: > > It seems to be an issue with the open jdk version: > > Expected feature release number in range of 9 to , but got: 15 => error

Re: Errored: apache/calcite#12834 (main - 94dc303)

2022-06-23 Thread Francis Chuang
It seems to be an issue with the open jdk version: Expected feature release number in range of 9 to , but got: 15 => error for OpenJDK 15 Expected feature release number in range of 9 to , but got: 18 => error for OpenJDK 18 On 24/06/2022 9:05 am, Julian Hyde wrote: Does anyone have any

Re: Errored: apache/calcite#12834 (main - 94dc303)

2022-06-23 Thread Julian Hyde
Does anyone have any idea why Travis failed for me? I pushed a one-line commit that had already succeeded in GitHub actions. Julian > On Jun 23, 2022, at 3:54 PM, Travis CI wrote: > > apache / calcite >

Re: PR Review Request

2022-06-23 Thread Julian Hyde
+1 to Stamatis’ idea. It won’t make things worse. :) But to repeat what I said earlier. We need existing committers to pull their weight. If necessary, committers need to talk to their managers and get time allocated to contribute to “housekeeping”. One important kind of housekeeping is

[jira] [Created] (CALCITE-5197) Bump gradle to 7.4.2 and add checksum autoupdate

2022-06-23 Thread Sergey Nuyanzin (Jira)
Sergey Nuyanzin created CALCITE-5197: Summary: Bump gradle to 7.4.2 and add checksum autoupdate Key: CALCITE-5197 URL: https://issues.apache.org/jira/browse/CALCITE-5197 Project: Calcite

[jira] [Created] (CALCITE-5196) Bump apiguardian version to 1.1.2 (latest)

2022-06-23 Thread Sergey Nuyanzin (Jira)
Sergey Nuyanzin created CALCITE-5196: Summary: Bump apiguardian version to 1.1.2 (latest) Key: CALCITE-5196 URL: https://issues.apache.org/jira/browse/CALCITE-5196 Project: Calcite Issue

Re: PR Review Request

2022-06-23 Thread Ruben Q L
+1 on Stamatis' idea, I think it could help with the current situation of lack of reviewers. Best, Ruben On Thu, Jun 23, 2022 at 12:56 PM Charles Givre wrote: > Hello all, > FWIW, If a committer/reviewer shortage is the issue, I'd second Stamatis's > recommendation. > Best, > -- C > > > On

Re: Aggregates push down

2022-06-23 Thread Benchao Li
Вадим, Yes, we are short of documentation at the moment. There is source code for adapters with processing aggregates, for example, > Mongodb, This is another different topic, adaptors such as Mongo/JDBC, they transformed all the RelNodes to their own convention, and then translate the RelNodes

Re: PR Review Request

2022-06-23 Thread Charles Givre
Hello all, FWIW, If a committer/reviewer shortage is the issue, I'd second Stamatis's recommendation. Best, -- C > On Jun 23, 2022, at 7:02 AM, Stamatis Zampetakis wrote: > > Hi all, > > How about granting Calcite committership to people who are already ASF > committers (in other projects)

Re: PR Review Request

2022-06-23 Thread Stamatis Zampetakis
Hi all, How about granting Calcite committership to people who are already ASF committers (in other projects) and they have a proven record of working with Calcite? Usually the PMC invites people to become committers to the project after having a few successful code contributions in

Re: CI Failed in JDK15

2022-06-23 Thread Alessandro Solimando
Hi Yanjing, we have had an identical failure in March (logs are not available anymore, unfortunately) in the snapshot build: "Build failed in Jenkins: Calcite » Calcite-snapshots #88", related to the commit "[fan_li_ya] [CALCITE-4976] Release Calcite 1.30.0" > Task :server:test > [0;31;1mFAILURE