Re: [DISCUSS] Towards Avatica 1.16.0

2019-12-04 Thread Francis Chuang

Hi everyone,

Just a quick reminder that I am making rc0 available for voting 
tomorrow. If you need to get any change in, or if the time-frame is 
inconvenient, please let me know.


Francis

On 2/12/2019 9:07 am, Francis Chuang wrote:
The Avatica 1.16.0 release seems to be a bit overdue. I plan to make rc0 
available for voting on Friday morning (the 6th of December) Australian 
Eastern Standard time (probably Thursday in some parts of Europe and 
America).


If there are any changes you would like to see in this release, please 
try to get them in. If committers could review and merge some of the 
open PRs[1], that would be highly appreciated too!


If the proposed time frame does not work for you, please let me know.

Francis

[1] https://github.com/apache/calcite-avatica/pulls

On 10/10/2019 10:26 am, Francis Chuang wrote:

Hey everyone,

It's been around 5 months since the last Avatica release. There's been 
a couple of commits to the code-base since and I'd like to use this as 
an opportunity to get a few more PRs in and make a release available 
for voting. I am happy to be release manager for this release.


In terms of PRs I am hoping for the following to be merged for this 
release:


- CALCITE-3384: Support Kerberos using SPNEGO over HTTPS - 
https://github.com/apache/calcite-avatica/pull/113 (Josh is reviewing)


- CALCITE-333: Add pluggable frame size limits - 
https://issues.apache.org/jira/browse/CALCITE- (looking for a 
reviewer)


- CALCITE-3199: Fix unixDateCeil and unixDateFloor - 
https://github.com/apache/calcite-avatica/pull/109 (looking for a 
reviewer)


- CALCITE-3162: Reading arrays from Calcite through JdbcMeta throws 
exception - https://github.com/apache/calcite-avatica/pull/106 
(looking for a reviewer)


- CALCITE-3163: Mapping types in AbstractCursor does not adhere to 
JDBC spec - https://github.com/apache/calcite-avatica/pull/105 
(looking for a reviewer)


- CALCITE-3158: Move build system to Gradle - 
https://github.com/apache/calcite-avatica/pull/104 (Vladimir has put a 
lot of effort into this. Can we get some consensus as to whether this 
should to be merged?)


- CALCITE-3078: Duplicate code for lastDay in Calcite and Avatica - 
https://github.com/apache/calcite-avatica/pull/98 (looking for a 
reviewer)


- No Jira: Support UNIX timestamp to string date - 
https://github.com/apache/calcite-avatica/pull/96 (can someone confirm 
if these changes are valid or should the PR be closed?)


- CALCITE-2704: Avoid using ISO-8859-1 to parse request in JSONHandler 
- https://github.com/apache/calcite-avatica/pull/85 (Someone posted 
some tests in the issue comments. Can someone please add the tests, 
review and merge?)


- CALCITE-1806: Add Spark JDBC tests - 
https://github.com/apache/calcite-avatica/pull/28 (Kevin, any chance 
you can take another look at this?)


If you have some spare cycles for Avatica, please pick up some of the 
PRs in this list, it would be very much appreciated!


Francis


Re: [DISCUSS] Towards Avatica 1.16.0

2019-12-01 Thread Francis Chuang
The Avatica 1.16.0 release seems to be a bit overdue. I plan to make rc0 
available for voting on Friday morning (the 6th of December) Australian 
Eastern Standard time (probably Thursday in some parts of Europe and 
America).


If there are any changes you would like to see in this release, please 
try to get them in. If committers could review and merge some of the 
open PRs[1], that would be highly appreciated too!


If the proposed time frame does not work for you, please let me know.

Francis

[1] https://github.com/apache/calcite-avatica/pulls

On 10/10/2019 10:26 am, Francis Chuang wrote:

Hey everyone,

It's been around 5 months since the last Avatica release. There's been a 
couple of commits to the code-base since and I'd like to use this as an 
opportunity to get a few more PRs in and make a release available for 
voting. I am happy to be release manager for this release.


In terms of PRs I am hoping for the following to be merged for this 
release:


- CALCITE-3384: Support Kerberos using SPNEGO over HTTPS - 
https://github.com/apache/calcite-avatica/pull/113 (Josh is reviewing)


- CALCITE-333: Add pluggable frame size limits - 
https://issues.apache.org/jira/browse/CALCITE- (looking for a reviewer)


- CALCITE-3199: Fix unixDateCeil and unixDateFloor - 
https://github.com/apache/calcite-avatica/pull/109 (looking for a reviewer)


- CALCITE-3162: Reading arrays from Calcite through JdbcMeta throws 
exception - https://github.com/apache/calcite-avatica/pull/106 (looking 
for a reviewer)


- CALCITE-3163: Mapping types in AbstractCursor does not adhere to JDBC 
spec - https://github.com/apache/calcite-avatica/pull/105 (looking for a 
reviewer)


- CALCITE-3158: Move build system to Gradle - 
https://github.com/apache/calcite-avatica/pull/104 (Vladimir has put a 
lot of effort into this. Can we get some consensus as to whether this 
should to be merged?)


- CALCITE-3078: Duplicate code for lastDay in Calcite and Avatica - 
https://github.com/apache/calcite-avatica/pull/98 (looking for a reviewer)


- No Jira: Support UNIX timestamp to string date - 
https://github.com/apache/calcite-avatica/pull/96 (can someone confirm 
if these changes are valid or should the PR be closed?)


- CALCITE-2704: Avoid using ISO-8859-1 to parse request in JSONHandler - 
https://github.com/apache/calcite-avatica/pull/85 (Someone posted some 
tests in the issue comments. Can someone please add the tests, review 
and merge?)


- CALCITE-1806: Add Spark JDBC tests - 
https://github.com/apache/calcite-avatica/pull/28 (Kevin, any chance you 
can take another look at this?)


If you have some spare cycles for Avatica, please pick up some of the 
PRs in this list, it would be very much appreciated!


Francis


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] https://github.com/apache/calcite/pull/1580 



On Nov 11, 2019, at 12:01 PM, Vladimir Sitnikov  
wrote:

The list you provide is outdated.


these plugins weren't added to the Gradle build at
some point.


Please feel free to execute "./gradlew tasks" and check what is available

To my best knowledge, the build script is good enough.

The missing items are

* "Unused dependency"
* "Used but undeclared dependency"


That, however, is not significant for Avatica since the number of
dependencies is low.
"unused dependency" and "used but undeclared" plugin for Gradle does not
exist yet.
We need "java-library" support, and that is a missing feature (
https://github.com/wfhartford/gradle-dependency-analyze/issues/39)

Vladimir





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:
> 
> The list you provide is outdated.
> 
>> these plugins weren't added to the Gradle build at
>> some point.
> 
> Please feel free to execute "./gradlew tasks" and check what is available
> 
> To my best knowledge, the build script is good enough.
> 
> The missing items are
>> * "Unused dependency"
>> * "Used but undeclared dependency"
> 
> That, however, is not significant for Avatica since the number of
> dependencies is low.
> "unused dependency" and "used but undeclared" plugin for Gradle does not
> exist yet.
> We need "java-library" support, and that is a missing feature (
> https://github.com/wfhartford/gradle-dependency-analyze/issues/39)
> 
> Vladimir



Re: [DISCUSS] Towards Avatica 1.16.0

2019-11-11 Thread Vladimir Sitnikov
The list you provide is outdated.

>these plugins weren't added to the Gradle build at
>some point.

Please feel free to execute "./gradlew tasks" and check what is available

To my best knowledge, the build script is good enough.

The missing items are
> * "Unused dependency"
> * "Used but undeclared dependency"

That, however, is not significant for Avatica since the number of
dependencies is low.
"unused dependency" and "used but undeclared" plugin for Gradle does not
exist yet.
We need "java-library" support, and that is a missing feature (
https://github.com/wfhartford/gradle-dependency-analyze/issues/39)

Vladimir


Re: [DISCUSS] Towards Avatica 1.16.0

2019-11-11 Thread Kevin Risden
>
> To my best knowledge, the missing bits are:
> * Documentation (site) update to reflect Maven -> Gradle
> * Errorprone
> * OWASP plugin
> * "Unused dependency"
> * "Used but undeclared dependency"
> * Removal of pom.xml files
> I'm not really sure there's a hard requirement to implement all of the
> above before flipping the switch.
> I'm inclined that OWASP and "unused/used dependency" can be implemented
> later. Freel free to correct me.


Are there Jiras to make sure these get added back in (specifically the
plugins to ensure build quality(? I think Gradle can be an improvement and
still learning my way around it. I know the above plugins were added
(mostly by me) to ensure that code quality doesn't suffer. It would be a
shame to regress since these plugins weren't added to the Gradle build at
some point.

Kevin Risden


On Sun, Nov 10, 2019 at 2:55 PM Vladimir Sitnikov <
sitnikov.vladi...@gmail.com> wrote:

> As a follow-up, I would like to thank everybody who helped to polish the
> change: Stamatis, Francis, et al.
>
> Vladimir
>


Re: [DISCUSS] Towards Avatica 1.16.0

2019-11-10 Thread Vladimir Sitnikov
As a follow-up, I would like to thank everybody who helped to polish the
change: Stamatis, Francis, et al.

Vladimir


Re: [DISCUSS] Towards Avatica 1.16.0

2019-11-10 Thread Vladimir Sitnikov
Ok, so I committed Maven -> Gradle change.

Even though I expect it to be ok, there might be glitches here and there.
It would be great if someone could verify if the generated pom.xml
files are good.

Note: Avatica pom.xml files are generated at build time, so they will be
different from those that existed in Git.

As a reminder, if you find yourself lost, try executing " ./gradlew tasks "
(if you forget tasks, try ./gradlew)
It should print relevant tasks, and I suggest we keep the most important
tasks there, and we hide less important ones.

The poms can be tested as follows:

"manual inspection":
$ ./gradlew generatePom && find . -name 'pom-default.xml'

./metrics/build/publications/metrics/pom-default.xml
./core/build/publications/core/pom-default.xml
./shaded/core/build/publications/avatica/pom-default.xml
./noop-driver/build/publications/noop-driver/pom-default.xml
./server/build/publications/server/pom-default.xml
./tck/build/publications/tck/pom-default.xml
./build/publications/avatica-parent/pom-default.xml
./build/publications/pluginMaven/pom-default.xml
./standalone-server/build/publications/standalone-server/pom-default.xml
./metrics-dropwizardmetrics/build/publications/metrics-dropwizardmetrics/pom-default.xml

"install to local Maven"
$ ./gradlew publishToMavenLocal

It will install Avatica artifacts to Maven local, so it could be tested
with other Maven projects.

Vladimir


Re: [DISCUSS] Towards Avatica 1.16.0

2019-11-07 Thread Vladimir Sitnikov
>I think it's also possible to get CALCITE-3158 (replace Maven with Gradle)

I think this is more or less ready to go.
The commits should probably be squashed, and I can do that if no-one else
plans to review the change.

There's "remove protobuf-generated files" change.
Is it truly needed to store protobuf-generated files in the Git sources?

PS. Apache JMeter has recently performed a release with Gradle.
Here's what the release manager says re Ant -> Gradle:
https://lists.apache.org/thread.html/c08780952b7fbd71eccff717cb494562723e11dc99ea1b4bb39af461@%3Cdev.jmeter.apache.org%3E

Vladimir


Re: [DISCUSS] Towards Avatica 1.16.0

2019-11-05 Thread Francis Chuang
I just wanted to follow up on the Avatica 1.16.0 release as I think it 
would be nice to get it out in the next couple of weeks.


Can someone please review the follow if they have got spare cycles?
- CALCITE-3401 https://github.com/apache/calcite-avatica/pull/115
- CALCITE- https://github.com/apache/calcite-avatica/pull/110
- CALCITE-3162 https://github.com/apache/calcite-avatica/pull/106
- CALCITE-3163 https://github.com/apache/calcite-avatica/pull/105

If CALCITE-2704 can be finished 
(https://github.com/apache/calcite-avatica/pull/85) it would be great too.


Can someone please see if 
https://github.com/apache/calcite-avatica/pull/96 is valid or should be 
closed?


I think it's also possible to get CALCITE-3158 (replace maven with 
Gradle) into this release (thanks, Vladimir!)


Francis

On 22/10/2019 9:31 am, Vladimir Sitnikov wrote:

Francis>files updated to switch over to using a Gradle docker container to
run
Francis>the build/release steps as well.

Docker does unify the environment, however, I'm not sure it is good to make
Docker the only option for the release.
Gradle enables single-command "build+test+publish to SVN/Nexus" workflow,
so I guess it might make docker.sh obsolete.

Could you give it a try?
Here are the commands:
https://github.com/vlsi/vlsi-release-plugins/tree/master/plugins/stage-vote-release-plugin#testing-release-procedure

Francis>If the change to move to Gradle is checked in

To my best knowledge, the missing bits are:
* Documentation (site) update to reflect Maven -> Gradle
* Errorprone
* OWASP plugin
* "Unused dependency"
* "Used but undeclared dependency"
* Removal of pom.xml files

I'm not really sure there's a hard requirement to implement all of the
above before flipping the switch.
I'm inclined that OWASP and "unused/used dependency" can be implemented
later. Freel free to correct me.

Vladimir



Re: [DISCUSS] Towards Avatica 1.16.0

2019-10-21 Thread Vladimir Sitnikov
Francis>files updated to switch over to using a Gradle docker container to
run
Francis>the build/release steps as well.

Docker does unify the environment, however, I'm not sure it is good to make
Docker the only option for the release.
Gradle enables single-command "build+test+publish to SVN/Nexus" workflow,
so I guess it might make docker.sh obsolete.

Could you give it a try?
Here are the commands:
https://github.com/vlsi/vlsi-release-plugins/tree/master/plugins/stage-vote-release-plugin#testing-release-procedure

Francis>If the change to move to Gradle is checked in

To my best knowledge, the missing bits are:
* Documentation (site) update to reflect Maven -> Gradle
* Errorprone
* OWASP plugin
* "Unused dependency"
* "Used but undeclared dependency"
* Removal of pom.xml files

I'm not really sure there's a hard requirement to implement all of the
above before flipping the switch.
I'm inclined that OWASP and "unused/used dependency" can be implemented
later. Freel free to correct me.

Vladimir


Re: [DISCUSS] Towards Avatica 1.16.0

2019-10-21 Thread Francis Chuang
I am happy to build the next release using Gradle (eventually we'll have 
to do it anyway if we are moving to Gradle).


I've previously committed a docker-compose file and shell script to 
automate the release:

- https://github.com/apache/calcite-avatica/blob/master/docker-compose.yml
- https://github.com/apache/calcite-avatica/blob/master/docker.sh

If the change to move to Gradle is checked in, I'd love to see those 2 
files updated to switch over to using a Gradle docker container to run 
the build/release steps as well.


Francis

On 22/10/2019 8:54 am, Stamatis Zampetakis wrote:

I think [CALCITE-3158] would be the main highlight of the next release. I
really like the feeling of Gradle and the idea of the mock ASF environment.

 From my tests so far everything seems good; I would like to verify that the
structure of the artifacts is good and then I think it can be merged.
Should be doable for November.

@Francis: How do you feel about preparing the next release via Gradle?

Best,
Stamatis

On Sun, Oct 20, 2019, 11:47 PM Francis Chuang 
wrote:


Thanks to those who have made some headway during the last week by
reviewing and merging PRs. Would early November be ample time for a rc0?
If more time is needed to review and merged the current PRs on Github,
please let me know.

Francis

On 11/10/2019 8:06 am, Francis Chuang wrote:

Thanks, Josh! Very much appreciated!

On 11/10/2019 2:35 am, Josh Elser wrote:

I'll try to get CALCITE-3384 in.

Will take a look through the reste here. A couple catch my eye.

On 10/9/19 7:26 PM, Francis Chuang wrote:

Hey everyone,

It's been around 5 months since the last Avatica release. There's
been a couple of commits to the code-base since and I'd like to use
this as an opportunity to get a few more PRs in and make a release
available for voting. I am happy to be release manager for this

release.


In terms of PRs I am hoping for the following to be merged for this
release:

- CALCITE-3384: Support Kerberos using SPNEGO over HTTPS -
https://github.com/apache/calcite-avatica/pull/113 (Josh is reviewing)

- CALCITE-333: Add pluggable frame size limits -
https://issues.apache.org/jira/browse/CALCITE- (looking for a
reviewer)

- CALCITE-3199: Fix unixDateCeil and unixDateFloor -
https://github.com/apache/calcite-avatica/pull/109 (looking for a
reviewer)

- CALCITE-3162: Reading arrays from Calcite through JdbcMeta throws
exception - https://github.com/apache/calcite-avatica/pull/106
(looking for a reviewer)

- CALCITE-3163: Mapping types in AbstractCursor does not adhere to
JDBC spec - https://github.com/apache/calcite-avatica/pull/105
(looking for a reviewer)

- CALCITE-3158: Move build system to Gradle -
https://github.com/apache/calcite-avatica/pull/104 (Vladimir has put
a lot of effort into this. Can we get some consensus as to whether
this should to be merged?)

- CALCITE-3078: Duplicate code for lastDay in Calcite and Avatica -
https://github.com/apache/calcite-avatica/pull/98 (looking for a
reviewer)

- No Jira: Support UNIX timestamp to string date -
https://github.com/apache/calcite-avatica/pull/96 (can someone
confirm if these changes are valid or should the PR be closed?)

- CALCITE-2704: Avoid using ISO-8859-1 to parse request in
JSONHandler - https://github.com/apache/calcite-avatica/pull/85
(Someone posted some tests in the issue comments. Can someone please
add the tests, review and merge?)

- CALCITE-1806: Add Spark JDBC tests -
https://github.com/apache/calcite-avatica/pull/28 (Kevin, any chance
you can take another look at this?)

If you have some spare cycles for Avatica, please pick up some of the
PRs in this list, it would be very much appreciated!

Francis






Re: [DISCUSS] Towards Avatica 1.16.0

2019-10-21 Thread Stamatis Zampetakis
I think [CALCITE-3158] would be the main highlight of the next release. I
really like the feeling of Gradle and the idea of the mock ASF environment.

>From my tests so far everything seems good; I would like to verify that the
structure of the artifacts is good and then I think it can be merged.
Should be doable for November.

@Francis: How do you feel about preparing the next release via Gradle?

Best,
Stamatis

On Sun, Oct 20, 2019, 11:47 PM Francis Chuang 
wrote:

> Thanks to those who have made some headway during the last week by
> reviewing and merging PRs. Would early November be ample time for a rc0?
> If more time is needed to review and merged the current PRs on Github,
> please let me know.
>
> Francis
>
> On 11/10/2019 8:06 am, Francis Chuang wrote:
> > Thanks, Josh! Very much appreciated!
> >
> > On 11/10/2019 2:35 am, Josh Elser wrote:
> >> I'll try to get CALCITE-3384 in.
> >>
> >> Will take a look through the reste here. A couple catch my eye.
> >>
> >> On 10/9/19 7:26 PM, Francis Chuang wrote:
> >>> Hey everyone,
> >>>
> >>> It's been around 5 months since the last Avatica release. There's
> >>> been a couple of commits to the code-base since and I'd like to use
> >>> this as an opportunity to get a few more PRs in and make a release
> >>> available for voting. I am happy to be release manager for this
> release.
> >>>
> >>> In terms of PRs I am hoping for the following to be merged for this
> >>> release:
> >>>
> >>> - CALCITE-3384: Support Kerberos using SPNEGO over HTTPS -
> >>> https://github.com/apache/calcite-avatica/pull/113 (Josh is reviewing)
> >>>
> >>> - CALCITE-333: Add pluggable frame size limits -
> >>> https://issues.apache.org/jira/browse/CALCITE- (looking for a
> >>> reviewer)
> >>>
> >>> - CALCITE-3199: Fix unixDateCeil and unixDateFloor -
> >>> https://github.com/apache/calcite-avatica/pull/109 (looking for a
> >>> reviewer)
> >>>
> >>> - CALCITE-3162: Reading arrays from Calcite through JdbcMeta throws
> >>> exception - https://github.com/apache/calcite-avatica/pull/106
> >>> (looking for a reviewer)
> >>>
> >>> - CALCITE-3163: Mapping types in AbstractCursor does not adhere to
> >>> JDBC spec - https://github.com/apache/calcite-avatica/pull/105
> >>> (looking for a reviewer)
> >>>
> >>> - CALCITE-3158: Move build system to Gradle -
> >>> https://github.com/apache/calcite-avatica/pull/104 (Vladimir has put
> >>> a lot of effort into this. Can we get some consensus as to whether
> >>> this should to be merged?)
> >>>
> >>> - CALCITE-3078: Duplicate code for lastDay in Calcite and Avatica -
> >>> https://github.com/apache/calcite-avatica/pull/98 (looking for a
> >>> reviewer)
> >>>
> >>> - No Jira: Support UNIX timestamp to string date -
> >>> https://github.com/apache/calcite-avatica/pull/96 (can someone
> >>> confirm if these changes are valid or should the PR be closed?)
> >>>
> >>> - CALCITE-2704: Avoid using ISO-8859-1 to parse request in
> >>> JSONHandler - https://github.com/apache/calcite-avatica/pull/85
> >>> (Someone posted some tests in the issue comments. Can someone please
> >>> add the tests, review and merge?)
> >>>
> >>> - CALCITE-1806: Add Spark JDBC tests -
> >>> https://github.com/apache/calcite-avatica/pull/28 (Kevin, any chance
> >>> you can take another look at this?)
> >>>
> >>> If you have some spare cycles for Avatica, please pick up some of the
> >>> PRs in this list, it would be very much appreciated!
> >>>
> >>> Francis
>


Re: [DISCUSS] Towards Avatica 1.16.0

2019-10-20 Thread Francis Chuang
Thanks to those who have made some headway during the last week by 
reviewing and merging PRs. Would early November be ample time for a rc0? 
If more time is needed to review and merged the current PRs on Github, 
please let me know.


Francis

On 11/10/2019 8:06 am, Francis Chuang wrote:

Thanks, Josh! Very much appreciated!

On 11/10/2019 2:35 am, Josh Elser wrote:

I'll try to get CALCITE-3384 in.

Will take a look through the reste here. A couple catch my eye.

On 10/9/19 7:26 PM, Francis Chuang wrote:

Hey everyone,

It's been around 5 months since the last Avatica release. There's 
been a couple of commits to the code-base since and I'd like to use 
this as an opportunity to get a few more PRs in and make a release 
available for voting. I am happy to be release manager for this release.


In terms of PRs I am hoping for the following to be merged for this 
release:


- CALCITE-3384: Support Kerberos using SPNEGO over HTTPS - 
https://github.com/apache/calcite-avatica/pull/113 (Josh is reviewing)


- CALCITE-333: Add pluggable frame size limits - 
https://issues.apache.org/jira/browse/CALCITE- (looking for a 
reviewer)


- CALCITE-3199: Fix unixDateCeil and unixDateFloor - 
https://github.com/apache/calcite-avatica/pull/109 (looking for a 
reviewer)


- CALCITE-3162: Reading arrays from Calcite through JdbcMeta throws 
exception - https://github.com/apache/calcite-avatica/pull/106 
(looking for a reviewer)


- CALCITE-3163: Mapping types in AbstractCursor does not adhere to 
JDBC spec - https://github.com/apache/calcite-avatica/pull/105 
(looking for a reviewer)


- CALCITE-3158: Move build system to Gradle - 
https://github.com/apache/calcite-avatica/pull/104 (Vladimir has put 
a lot of effort into this. Can we get some consensus as to whether 
this should to be merged?)


- CALCITE-3078: Duplicate code for lastDay in Calcite and Avatica - 
https://github.com/apache/calcite-avatica/pull/98 (looking for a 
reviewer)


- No Jira: Support UNIX timestamp to string date - 
https://github.com/apache/calcite-avatica/pull/96 (can someone 
confirm if these changes are valid or should the PR be closed?)


- CALCITE-2704: Avoid using ISO-8859-1 to parse request in 
JSONHandler - https://github.com/apache/calcite-avatica/pull/85 
(Someone posted some tests in the issue comments. Can someone please 
add the tests, review and merge?)


- CALCITE-1806: Add Spark JDBC tests - 
https://github.com/apache/calcite-avatica/pull/28 (Kevin, any chance 
you can take another look at this?)


If you have some spare cycles for Avatica, please pick up some of the 
PRs in this list, it would be very much appreciated!


Francis


Re: [DISCUSS] Towards Avatica 1.16.0

2019-10-10 Thread Francis Chuang

Thanks, Josh! Very much appreciated!

On 11/10/2019 2:35 am, Josh Elser wrote:

I'll try to get CALCITE-3384 in.

Will take a look through the reste here. A couple catch my eye.

On 10/9/19 7:26 PM, Francis Chuang wrote:

Hey everyone,

It's been around 5 months since the last Avatica release. There's been 
a couple of commits to the code-base since and I'd like to use this as 
an opportunity to get a few more PRs in and make a release available 
for voting. I am happy to be release manager for this release.


In terms of PRs I am hoping for the following to be merged for this 
release:


- CALCITE-3384: Support Kerberos using SPNEGO over HTTPS - 
https://github.com/apache/calcite-avatica/pull/113 (Josh is reviewing)


- CALCITE-333: Add pluggable frame size limits - 
https://issues.apache.org/jira/browse/CALCITE- (looking for a 
reviewer)


- CALCITE-3199: Fix unixDateCeil and unixDateFloor - 
https://github.com/apache/calcite-avatica/pull/109 (looking for a 
reviewer)


- CALCITE-3162: Reading arrays from Calcite through JdbcMeta throws 
exception - https://github.com/apache/calcite-avatica/pull/106 
(looking for a reviewer)


- CALCITE-3163: Mapping types in AbstractCursor does not adhere to 
JDBC spec - https://github.com/apache/calcite-avatica/pull/105 
(looking for a reviewer)


- CALCITE-3158: Move build system to Gradle - 
https://github.com/apache/calcite-avatica/pull/104 (Vladimir has put a 
lot of effort into this. Can we get some consensus as to whether this 
should to be merged?)


- CALCITE-3078: Duplicate code for lastDay in Calcite and Avatica - 
https://github.com/apache/calcite-avatica/pull/98 (looking for a 
reviewer)


- No Jira: Support UNIX timestamp to string date - 
https://github.com/apache/calcite-avatica/pull/96 (can someone confirm 
if these changes are valid or should the PR be closed?)


- CALCITE-2704: Avoid using ISO-8859-1 to parse request in JSONHandler 
- https://github.com/apache/calcite-avatica/pull/85 (Someone posted 
some tests in the issue comments. Can someone please add the tests, 
review and merge?)


- CALCITE-1806: Add Spark JDBC tests - 
https://github.com/apache/calcite-avatica/pull/28 (Kevin, any chance 
you can take another look at this?)


If you have some spare cycles for Avatica, please pick up some of the 
PRs in this list, it would be very much appreciated!


Francis


Re: [DISCUSS] Towards Avatica 1.16.0

2019-10-10 Thread Josh Elser

I'll try to get CALCITE-3384 in.

Will take a look through the reste here. A couple catch my eye.

On 10/9/19 7:26 PM, Francis Chuang wrote:

Hey everyone,

It's been around 5 months since the last Avatica release. There's been a 
couple of commits to the code-base since and I'd like to use this as an 
opportunity to get a few more PRs in and make a release available for 
voting. I am happy to be release manager for this release.


In terms of PRs I am hoping for the following to be merged for this 
release:


- CALCITE-3384: Support Kerberos using SPNEGO over HTTPS - 
https://github.com/apache/calcite-avatica/pull/113 (Josh is reviewing)


- CALCITE-333: Add pluggable frame size limits - 
https://issues.apache.org/jira/browse/CALCITE- (looking for a reviewer)


- CALCITE-3199: Fix unixDateCeil and unixDateFloor - 
https://github.com/apache/calcite-avatica/pull/109 (looking for a reviewer)


- CALCITE-3162: Reading arrays from Calcite through JdbcMeta throws 
exception - https://github.com/apache/calcite-avatica/pull/106 (looking 
for a reviewer)


- CALCITE-3163: Mapping types in AbstractCursor does not adhere to JDBC 
spec - https://github.com/apache/calcite-avatica/pull/105 (looking for a 
reviewer)


- CALCITE-3158: Move build system to Gradle - 
https://github.com/apache/calcite-avatica/pull/104 (Vladimir has put a 
lot of effort into this. Can we get some consensus as to whether this 
should to be merged?)


- CALCITE-3078: Duplicate code for lastDay in Calcite and Avatica - 
https://github.com/apache/calcite-avatica/pull/98 (looking for a reviewer)


- No Jira: Support UNIX timestamp to string date - 
https://github.com/apache/calcite-avatica/pull/96 (can someone confirm 
if these changes are valid or should the PR be closed?)


- CALCITE-2704: Avoid using ISO-8859-1 to parse request in JSONHandler - 
https://github.com/apache/calcite-avatica/pull/85 (Someone posted some 
tests in the issue comments. Can someone please add the tests, review 
and merge?)


- CALCITE-1806: Add Spark JDBC tests - 
https://github.com/apache/calcite-avatica/pull/28 (Kevin, any chance you 
can take another look at this?)


If you have some spare cycles for Avatica, please pick up some of the 
PRs in this list, it would be very much appreciated!


Francis


Re: [DISCUSS] Towards Avatica 1.16.0

2019-10-09 Thread Haisheng Yuan
I would like to see CALCITE-3158 get merged. But it seems to be still working 
in progress.

- Haisheng

--
发件人:Francis Chuang
日 期:2019年10月10日 07:26:01
收件人:
主 题:[DISCUSS] Towards Avatica 1.16.0

Hey everyone,

It's been around 5 months since the last Avatica release. There's been a 
couple of commits to the code-base since and I'd like to use this as an 
opportunity to get a few more PRs in and make a release available for 
voting. I am happy to be release manager for this release.

In terms of PRs I am hoping for the following to be merged for this release:

- CALCITE-3384: Support Kerberos using SPNEGO over HTTPS - 
https://github.com/apache/calcite-avatica/pull/113 (Josh is reviewing)

- CALCITE-333: Add pluggable frame size limits - 
https://issues.apache.org/jira/browse/CALCITE- (looking for a reviewer)

- CALCITE-3199: Fix unixDateCeil and unixDateFloor - 
https://github.com/apache/calcite-avatica/pull/109 (looking for a reviewer)

- CALCITE-3162: Reading arrays from Calcite through JdbcMeta throws 
exception - https://github.com/apache/calcite-avatica/pull/106 (looking 
for a reviewer)

- CALCITE-3163: Mapping types in AbstractCursor does not adhere to JDBC 
spec - https://github.com/apache/calcite-avatica/pull/105 (looking for a 
reviewer)

- CALCITE-3158: Move build system to Gradle - 
https://github.com/apache/calcite-avatica/pull/104 (Vladimir has put a 
lot of effort into this. Can we get some consensus as to whether this 
should to be merged?)

- CALCITE-3078: Duplicate code for lastDay in Calcite and Avatica - 
https://github.com/apache/calcite-avatica/pull/98 (looking for a reviewer)

- No Jira: Support UNIX timestamp to string date - 
https://github.com/apache/calcite-avatica/pull/96 (can someone confirm 
if these changes are valid or should the PR be closed?)

- CALCITE-2704: Avoid using ISO-8859-1 to parse request in JSONHandler - 
https://github.com/apache/calcite-avatica/pull/85 (Someone posted some 
tests in the issue comments. Can someone please add the tests, review 
and merge?)

- CALCITE-1806: Add Spark JDBC tests - 
https://github.com/apache/calcite-avatica/pull/28 (Kevin, any chance you 
can take another look at this?)

If you have some spare cycles for Avatica, please pick up some of the 
PRs in this list, it would be very much appreciated!

Francis


[DISCUSS] Towards Avatica 1.16.0

2019-10-09 Thread Francis Chuang

Hey everyone,

It's been around 5 months since the last Avatica release. There's been a 
couple of commits to the code-base since and I'd like to use this as an 
opportunity to get a few more PRs in and make a release available for 
voting. I am happy to be release manager for this release.


In terms of PRs I am hoping for the following to be merged for this release:

- CALCITE-3384: Support Kerberos using SPNEGO over HTTPS - 
https://github.com/apache/calcite-avatica/pull/113 (Josh is reviewing)


- CALCITE-333: Add pluggable frame size limits - 
https://issues.apache.org/jira/browse/CALCITE- (looking for a reviewer)


- CALCITE-3199: Fix unixDateCeil and unixDateFloor - 
https://github.com/apache/calcite-avatica/pull/109 (looking for a reviewer)


- CALCITE-3162: Reading arrays from Calcite through JdbcMeta throws 
exception - https://github.com/apache/calcite-avatica/pull/106 (looking 
for a reviewer)


- CALCITE-3163: Mapping types in AbstractCursor does not adhere to JDBC 
spec - https://github.com/apache/calcite-avatica/pull/105 (looking for a 
reviewer)


- CALCITE-3158: Move build system to Gradle - 
https://github.com/apache/calcite-avatica/pull/104 (Vladimir has put a 
lot of effort into this. Can we get some consensus as to whether this 
should to be merged?)


- CALCITE-3078: Duplicate code for lastDay in Calcite and Avatica - 
https://github.com/apache/calcite-avatica/pull/98 (looking for a reviewer)


- No Jira: Support UNIX timestamp to string date - 
https://github.com/apache/calcite-avatica/pull/96 (can someone confirm 
if these changes are valid or should the PR be closed?)


- CALCITE-2704: Avoid using ISO-8859-1 to parse request in JSONHandler - 
https://github.com/apache/calcite-avatica/pull/85 (Someone posted some 
tests in the issue comments. Can someone please add the tests, review 
and merge?)


- CALCITE-1806: Add Spark JDBC tests - 
https://github.com/apache/calcite-avatica/pull/28 (Kevin, any chance you 
can take another look at this?)


If you have some spare cycles for Avatica, please pick up some of the 
PRs in this list, it would be very much appreciated!


Francis