Re: Guava version

2021-09-21 Thread Julian Hyde
Thank you! That worked. I am close to a solution in https://github.com/julianhyde/calcite/tree/4789-guava-19 . Reviews welcome. > On Sep 21, 2021, at 8:24 PM, James Starr wrote: > > I believe you want ./gradlew -Pguava.version=19.0.

Re: Tableau dialect specification for Calcite

2021-09-21 Thread Laurent Goujon
We will get back to you guys! Laurent On Mon, Sep 20, 2021 at 4:55 PM Julian Hyde wrote: > Looks very promising, and has the .tdd file that I was looking for. Thanks > Jacques. > > That said, there is no LICENSE file. I will file an issue to clarify the > license. > > Julian > > > > > On Sep

Re: Guava version

2021-09-21 Thread James Starr
I believe you want ./gradlew -Pguava.version=19.0. James On Tue, Sep 21, 2021 at 4:09 PM Julian Hyde wrote: > How do I set the Guava version from the Gradle command line? > > I’d expect > > ./gradlew -Dguava.version=19.0 > > to work but it doesn’t. > > (We used to test the supported range

[DISCUSS] Changing from ImmutableBeans to Immutables

2021-09-21 Thread Jacques Nadeau
In the original creation of ImmutableBeans [1], there was a discussion of using Immutables instead of something custom to do the configuration pojos. Because no one did the work of integrating one of those off the shelf packages at the time, we just went forward with the custom ImmutableBeans.

[jira] [Created] (CALCITE-4789) Build is broken on Guava versions < 21

2021-09-21 Thread Julian Hyde (Jira)
Julian Hyde created CALCITE-4789: Summary: Build is broken on Guava versions < 21 Key: CALCITE-4789 URL: https://issues.apache.org/jira/browse/CALCITE-4789 Project: Calcite Issue Type: Bug

Guava version

2021-09-21 Thread Julian Hyde
How do I set the Guava version from the Gradle command line? I’d expect ./gradlew -Dguava.version=19.0 to work but it doesn’t. (We used to test the supported range of Guava versions in CI. Then we stopped because Gradle. And now it turns out that we’re broken on Gradle < 21.0.) Julian

Re: [DISCUSS] Pronouns

2021-09-21 Thread Michael Mior
+1 Thanks Julian! I'll add that in many cases, I see the possessive pronoun (e.g. his) also included but I don't know enough about the reasons for this to feel strongly either way. I've added my pronouns as well. -- Michael Mior mm...@apache.org Le mar. 21 sept. 2021 à 14:16, Julian Hyde a

Re: Inconsistency of javadoc and actual behaviour of RelNode#getVariablesSet

2021-09-21 Thread Konstantin Orlov
Hi, Julian Thanks for your reply! > I believe that variables can only be set in the current RelNode. (Read a row > from input, set the variable, then evaluate a Rex expression or restart the > right input. It’s like a ‘for’ loop.) Actually, this is in line with my expectations. So, I'll file