Re: [DISCUSS] Do we need a Avatica release before Calcite 1.29.0 release?

2021-12-13 Thread Julian Hyde
Correct.

On Mon, Dec 13, 2021 at 5:24 PM Rui Wang  wrote:
>
> Ok to confirm, as https://issues.apache.org/jira/browse/CALCITE-4937 is
> marked as resolved, there is no blocker from Avatica side for 1.29.0
> release?
>
>
> -Rui
>
> On Fri, Dec 10, 2021 at 1:47 PM Jacques Nadeau  wrote:
>
> > Given the conversation around the log4j cve, it seems like we may need an
> > Avatica release afterall...
> >
> > Let's see how that discussion resolves.
> >
> > On Thu, Dec 9, 2021 at 12:47 PM Rui Wang  wrote:
> >
> > > Ack thanks!
> > >
> > > If there is no objection until the end of this week I will just go for
> > the
> > > 1.29.0 release process.
> > >
> > >
> > > -Rui
> > >
> > > On Thu, Dec 9, 2021 at 11:40 AM Josh Elser  wrote:
> > >
> > > > CALCITE-4152 is a big performance improvement to Avatica with
> > > > SPNEGO/Kerberos, but not at all about correctness/datatypes that
> > Calcite
> > > > needs.
> > > >
> > > > I'd agree with Julian.
> > > >
> > > > On 12/8/21 7:03 PM, Julian Hyde wrote:
> > > > > Probably not.
> > > > >
> > > > > * 4cf769f8e - (HEAD -> master, origin/master, origin/HEAD) Disable
> > > > Travis job that uses Calcite master until [CALCITE-4877] is fixed (4
> > > weeks
> > > > ago) 
> > > > > * 434cf002f - [CALCITE-4877] Make the exception information more
> > > > explicit for instantiate plugin. (5 weeks ago) 
> > > > > * 57c254f24 - [CALCITE-4840] Make README easier to scan (6 weeks ago)
> > > > 
> > > > > * c3a919234 - [CALCITE-4152] Upgrade Avatica to use the configurable
> > > > SPNEGO Jetty implementation (6 weeks ago) 
> > > > > * 95c1eb06e - [CALCITE-4828] Standard exception console output (7
> > weeks
> > > > ago) 
> > > > > * e462b09d4 - [CALCITE-4837] FLOOR/CEIL for DECADE, CENTURY,
> > MILLENIUM
> > > > return wrong results (8 weeks ago) 
> > > > > * ef277ff34 - Update website for Avatica 1.19.0 release (8 weeks ago)
> > > > 
> > > > >
> > > > >> On Dec 8, 2021, at 2:37 PM, Rui Wang  wrote:
> > > > >>
> > > > >> Hi community,
> > > > >>
> > > > >> Do you think we need an Avatica release before Calcite 1.29.0?
> > > > >>
> > > > >>
> > > > >> -Rui
> > > > >
> > > >
> > >
> >


Re: [DISCUSS] Do we need a Avatica release before Calcite 1.29.0 release?

2021-12-13 Thread Rui Wang
Ok to confirm, as https://issues.apache.org/jira/browse/CALCITE-4937 is
marked as resolved, there is no blocker from Avatica side for 1.29.0
release?


-Rui

On Fri, Dec 10, 2021 at 1:47 PM Jacques Nadeau  wrote:

> Given the conversation around the log4j cve, it seems like we may need an
> Avatica release afterall...
>
> Let's see how that discussion resolves.
>
> On Thu, Dec 9, 2021 at 12:47 PM Rui Wang  wrote:
>
> > Ack thanks!
> >
> > If there is no objection until the end of this week I will just go for
> the
> > 1.29.0 release process.
> >
> >
> > -Rui
> >
> > On Thu, Dec 9, 2021 at 11:40 AM Josh Elser  wrote:
> >
> > > CALCITE-4152 is a big performance improvement to Avatica with
> > > SPNEGO/Kerberos, but not at all about correctness/datatypes that
> Calcite
> > > needs.
> > >
> > > I'd agree with Julian.
> > >
> > > On 12/8/21 7:03 PM, Julian Hyde wrote:
> > > > Probably not.
> > > >
> > > > * 4cf769f8e - (HEAD -> master, origin/master, origin/HEAD) Disable
> > > Travis job that uses Calcite master until [CALCITE-4877] is fixed (4
> > weeks
> > > ago) 
> > > > * 434cf002f - [CALCITE-4877] Make the exception information more
> > > explicit for instantiate plugin. (5 weeks ago) 
> > > > * 57c254f24 - [CALCITE-4840] Make README easier to scan (6 weeks ago)
> > > 
> > > > * c3a919234 - [CALCITE-4152] Upgrade Avatica to use the configurable
> > > SPNEGO Jetty implementation (6 weeks ago) 
> > > > * 95c1eb06e - [CALCITE-4828] Standard exception console output (7
> weeks
> > > ago) 
> > > > * e462b09d4 - [CALCITE-4837] FLOOR/CEIL for DECADE, CENTURY,
> MILLENIUM
> > > return wrong results (8 weeks ago) 
> > > > * ef277ff34 - Update website for Avatica 1.19.0 release (8 weeks ago)
> > > 
> > > >
> > > >> On Dec 8, 2021, at 2:37 PM, Rui Wang  wrote:
> > > >>
> > > >> Hi community,
> > > >>
> > > >> Do you think we need an Avatica release before Calcite 1.29.0?
> > > >>
> > > >>
> > > >> -Rui
> > > >
> > >
> >
>


Re: [DISCUSS] Do we need a Avatica release before Calcite 1.29.0 release?

2021-12-10 Thread Jacques Nadeau
Given the conversation around the log4j cve, it seems like we may need an
Avatica release afterall...

Let's see how that discussion resolves.

On Thu, Dec 9, 2021 at 12:47 PM Rui Wang  wrote:

> Ack thanks!
>
> If there is no objection until the end of this week I will just go for the
> 1.29.0 release process.
>
>
> -Rui
>
> On Thu, Dec 9, 2021 at 11:40 AM Josh Elser  wrote:
>
> > CALCITE-4152 is a big performance improvement to Avatica with
> > SPNEGO/Kerberos, but not at all about correctness/datatypes that Calcite
> > needs.
> >
> > I'd agree with Julian.
> >
> > On 12/8/21 7:03 PM, Julian Hyde wrote:
> > > Probably not.
> > >
> > > * 4cf769f8e - (HEAD -> master, origin/master, origin/HEAD) Disable
> > Travis job that uses Calcite master until [CALCITE-4877] is fixed (4
> weeks
> > ago) 
> > > * 434cf002f - [CALCITE-4877] Make the exception information more
> > explicit for instantiate plugin. (5 weeks ago) 
> > > * 57c254f24 - [CALCITE-4840] Make README easier to scan (6 weeks ago)
> > 
> > > * c3a919234 - [CALCITE-4152] Upgrade Avatica to use the configurable
> > SPNEGO Jetty implementation (6 weeks ago) 
> > > * 95c1eb06e - [CALCITE-4828] Standard exception console output (7 weeks
> > ago) 
> > > * e462b09d4 - [CALCITE-4837] FLOOR/CEIL for DECADE, CENTURY, MILLENIUM
> > return wrong results (8 weeks ago) 
> > > * ef277ff34 - Update website for Avatica 1.19.0 release (8 weeks ago)
> > 
> > >
> > >> On Dec 8, 2021, at 2:37 PM, Rui Wang  wrote:
> > >>
> > >> Hi community,
> > >>
> > >> Do you think we need an Avatica release before Calcite 1.29.0?
> > >>
> > >>
> > >> -Rui
> > >
> >
>


Re: [DISCUSS] Do we need a Avatica release before Calcite 1.29.0 release?

2021-12-09 Thread Rui Wang
Ack thanks!

If there is no objection until the end of this week I will just go for the
1.29.0 release process.


-Rui

On Thu, Dec 9, 2021 at 11:40 AM Josh Elser  wrote:

> CALCITE-4152 is a big performance improvement to Avatica with
> SPNEGO/Kerberos, but not at all about correctness/datatypes that Calcite
> needs.
>
> I'd agree with Julian.
>
> On 12/8/21 7:03 PM, Julian Hyde wrote:
> > Probably not.
> >
> > * 4cf769f8e - (HEAD -> master, origin/master, origin/HEAD) Disable
> Travis job that uses Calcite master until [CALCITE-4877] is fixed (4 weeks
> ago) 
> > * 434cf002f - [CALCITE-4877] Make the exception information more
> explicit for instantiate plugin. (5 weeks ago) 
> > * 57c254f24 - [CALCITE-4840] Make README easier to scan (6 weeks ago)
> 
> > * c3a919234 - [CALCITE-4152] Upgrade Avatica to use the configurable
> SPNEGO Jetty implementation (6 weeks ago) 
> > * 95c1eb06e - [CALCITE-4828] Standard exception console output (7 weeks
> ago) 
> > * e462b09d4 - [CALCITE-4837] FLOOR/CEIL for DECADE, CENTURY, MILLENIUM
> return wrong results (8 weeks ago) 
> > * ef277ff34 - Update website for Avatica 1.19.0 release (8 weeks ago)
> 
> >
> >> On Dec 8, 2021, at 2:37 PM, Rui Wang  wrote:
> >>
> >> Hi community,
> >>
> >> Do you think we need an Avatica release before Calcite 1.29.0?
> >>
> >>
> >> -Rui
> >
>


Re: [DISCUSS] Do we need a Avatica release before Calcite 1.29.0 release?

2021-12-09 Thread Josh Elser
CALCITE-4152 is a big performance improvement to Avatica with 
SPNEGO/Kerberos, but not at all about correctness/datatypes that Calcite 
needs.


I'd agree with Julian.

On 12/8/21 7:03 PM, Julian Hyde wrote:

Probably not.

* 4cf769f8e - (HEAD -> master, origin/master, origin/HEAD) Disable Travis job that 
uses Calcite master until [CALCITE-4877] is fixed (4 weeks ago) 
* 434cf002f - [CALCITE-4877] Make the exception information more explicit for 
instantiate plugin. (5 weeks ago) 
* 57c254f24 - [CALCITE-4840] Make README easier to scan (6 weeks ago) 
* c3a919234 - [CALCITE-4152] Upgrade Avatica to use the configurable SPNEGO Jetty 
implementation (6 weeks ago) 
* 95c1eb06e - [CALCITE-4828] Standard exception console output (7 weeks ago) 

* e462b09d4 - [CALCITE-4837] FLOOR/CEIL for DECADE, CENTURY, MILLENIUM return wrong 
results (8 weeks ago) 
* ef277ff34 - Update website for Avatica 1.19.0 release (8 weeks ago) 


On Dec 8, 2021, at 2:37 PM, Rui Wang  wrote:

Hi community,

Do you think we need an Avatica release before Calcite 1.29.0?


-Rui




Re: [DISCUSS] Do we need a Avatica release before Calcite 1.29.0 release?

2021-12-08 Thread Julian Hyde
Probably not.

* 4cf769f8e - (HEAD -> master, origin/master, origin/HEAD) Disable Travis job 
that uses Calcite master until [CALCITE-4877] is fixed (4 weeks ago) 
* 434cf002f - [CALCITE-4877] Make the exception information more explicit for 
instantiate plugin. (5 weeks ago) 
* 57c254f24 - [CALCITE-4840] Make README easier to scan (6 weeks ago) 
* c3a919234 - [CALCITE-4152] Upgrade Avatica to use the configurable SPNEGO 
Jetty implementation (6 weeks ago) 
* 95c1eb06e - [CALCITE-4828] Standard exception console output (7 weeks ago) 

* e462b09d4 - [CALCITE-4837] FLOOR/CEIL for DECADE, CENTURY, MILLENIUM return 
wrong results (8 weeks ago) 
* ef277ff34 - Update website for Avatica 1.19.0 release (8 weeks ago) 

> On Dec 8, 2021, at 2:37 PM, Rui Wang  wrote:
> 
> Hi community,
> 
> Do you think we need an Avatica release before Calcite 1.29.0?
> 
> 
> -Rui



[DISCUSS] Do we need a Avatica release before Calcite 1.29.0 release?

2021-12-08 Thread Rui Wang
Hi community,

Do you think we need an Avatica release before Calcite 1.29.0?


-Rui