[GitHub] calcite-avatica-go issue #19: Fix readme

2018-04-16 Thread hsyuan
Github user hsyuan commented on the issue: https://github.com/apache/calcite-avatica-go/pull/19 So you don't need to open a PR, just commit it. ---

[GitHub] calcite-avatica-go pull request #19: Fix readme

2018-04-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/calcite-avatica-go/pull/19 ---

[GitHub] calcite-avatica-go pull request #19: Fix readme

2018-04-16 Thread F21
GitHub user F21 opened a pull request: https://github.com/apache/calcite-avatica-go/pull/19 Fix readme You can merge this pull request into a Git repository by running: $ git pull https://github.com/Boostport/calcite-avatica-go fix-readme Alternatively you can review and

Re: Towards Avatica-Go release ?.?

2018-04-16 Thread Francis Chuang
Julian, The release (3.0.0-rc0) is ready for voting. Do the artifacts need to be uploaded before sending out the release email? If so, can you please sign them? Instructions for making and signing release artifacts are here:

[GitHub] calcite-avatica-go pull request #18: Fix link in release history

2018-04-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/calcite-avatica-go/pull/18 ---

[GitHub] calcite-avatica-go pull request #18: Fix link in release history

2018-04-16 Thread F21
GitHub user F21 opened a pull request: https://github.com/apache/calcite-avatica-go/pull/18 Fix link in release history You can merge this pull request into a Git repository by running: $ git pull https://github.com/Boostport/calcite-avatica-go fix-link Alternatively you can

[GitHub] calcite-avatica-go pull request #2: Update readme and move documentation int...

2018-04-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/calcite-avatica-go/pull/2 ---

[GitHub] calcite-avatica-go pull request #17: Update dependencies

2018-04-16 Thread F21
Github user F21 closed the pull request at: https://github.com/apache/calcite-avatica-go/pull/17 ---

[GitHub] calcite-avatica-go pull request #17: Update dependencies

2018-04-16 Thread F21
GitHub user F21 opened a pull request: https://github.com/apache/calcite-avatica-go/pull/17 Update dependencies You can merge this pull request into a Git repository by running: $ git pull https://github.com/Boostport/calcite-avatica-go update-deps Alternatively you can

[GitHub] calcite-avatica-go pull request #16: Remove go-cleanhttp

2018-04-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/calcite-avatica-go/pull/16 ---

[GitHub] calcite-avatica-go pull request #16: Remove go-cleanhttp

2018-04-16 Thread F21
Github user F21 closed the pull request at: https://github.com/apache/calcite-avatica-go/pull/16 ---

[GitHub] calcite-avatica-go pull request #16: Remove go-cleanhttp

2018-04-16 Thread F21
GitHub user F21 opened a pull request: https://github.com/apache/calcite-avatica-go/pull/16 Remove go-cleanhttp You can merge this pull request into a Git repository by running: $ git pull https://github.com/Boostport/calcite-avatica-go remove-cleanhttp Alternatively you can

Re: Supporting named row construction in Calcite SQL

2018-04-16 Thread Julian Hyde
Yes. “Ask Julian”. The picture is in a Google Slides document. I don’t want to share it publicly, and I don’t want to check in a massive .SVG file. I don’t think it’s worth the effort to make this small part of the process scalable. > On Apr 16, 2018, at 5:22 PM, Shuyi Chen

[GitHub] calcite-avatica-go pull request #15: Add script to generate release artifact...

2018-04-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/calcite-avatica-go/pull/15 ---

Re: Supporting named row construction in Calcite SQL

2018-04-16 Thread Shuyi Chen
Thanks a lot, Julian. Do we have instructions on how to update the website and the picture documented? On Mon, Apr 16, 2018 at 4:42 AM, Michael Mior wrote: > Thanks Julian! > > -- > Michael Mior > mm...@apache.org > > -- > Michael Mior > mm...@uwaterloo.ca > > 2018-04-16

[GitHub] calcite-avatica-go pull request #15: Add script to generate release artifact...

2018-04-16 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica-go/pull/15#discussion_r181917580 --- Diff: make-release-artifacts.sh --- @@ -0,0 +1,26 @@ +# Clean dist directory +rm -rf dist +mkdir -p dist + +# Get new tags

[GitHub] calcite-avatica-go pull request #15: Add script to generate release artifact...

2018-04-16 Thread F21
Github user F21 commented on a diff in the pull request: https://github.com/apache/calcite-avatica-go/pull/15#discussion_r181917310 --- Diff: make-release-artifacts.sh --- @@ -0,0 +1,26 @@ +# Clean dist directory +rm -rf dist +mkdir -p dist + +# Get new tags

[GitHub] calcite-avatica-go pull request #15: Add script to generate release artifact...

2018-04-16 Thread F21
Github user F21 commented on a diff in the pull request: https://github.com/apache/calcite-avatica-go/pull/15#discussion_r181917109 --- Diff: make-release-artifacts.sh --- @@ -0,0 +1,26 @@ +# Clean dist directory +rm -rf dist +mkdir -p dist + +# Get new tags

[GitHub] calcite-avatica-go pull request #15: Add script to generate release artifact...

2018-04-16 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica-go/pull/15#discussion_r181916510 --- Diff: make-release-artifacts.sh --- @@ -0,0 +1,26 @@ +# Clean dist directory +rm -rf dist +mkdir -p dist + +# Get new tags

[GitHub] calcite-avatica-go pull request #15: Add script to generate release artifact...

2018-04-16 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica-go/pull/15#discussion_r181916445 --- Diff: make-release-artifacts.sh --- @@ -0,0 +1,26 @@ +# Clean dist directory +rm -rf dist +mkdir -p dist + +# Get new tags

[GitHub] calcite-avatica-go pull request #15: Add script to generate release artifact...

2018-04-16 Thread F21
GitHub user F21 opened a pull request: https://github.com/apache/calcite-avatica-go/pull/15 Add script to generate release artifacts You can merge this pull request into a Git repository by running: $ git pull https://github.com/Boostport/calcite-avatica-go

[jira] [Created] (CALCITE-2260) RelFieldTrimmer incorrectly trims fields when trimming a distinct UNION

2018-04-16 Thread Jacques Nadeau (JIRA)
Jacques Nadeau created CALCITE-2260: --- Summary: RelFieldTrimmer incorrectly trims fields when trimming a distinct UNION Key: CALCITE-2260 URL: https://issues.apache.org/jira/browse/CALCITE-2260

Re: Supporting named row construction in Calcite SQL

2018-04-16 Thread Michael Mior
Thanks Julian! -- Michael Mior mm...@apache.org -- Michael Mior mm...@uwaterloo.ca 2018-04-16 2:26 GMT-04:00 Julian Hyde : > Beam and HerdDB are now on “powered by”: https://calcite.apache.org/ > docs/powered_by.html > > > On

Re: Towards Avatica-Go release ?.?

2018-04-16 Thread Francis Chuang
Thanks, Julian. I will add a release script to the repo and add the release notes to the site/ directory. I also had to make some breaking changes to the public interfaces with the updates, so we'll release it as 3.0.0. Francis On 16/04/2018 4:44 PM, Julian Hyde wrote: Yes, the script

Re: Towards Avatica-Go release ?.?

2018-04-16 Thread Julian Hyde
Yes, the script isn’t very complicated. We probably need half a dozen lines of shell script to create the tar.gz file, sign it, and generate .sha256 checksums. I don’t mind where you put those lines of shell script, as long as the next RM can find them. The site looks good. It doesn’t have to

Re: Towards Avatica-Go release ?.?

2018-04-16 Thread F21
Hey Julian, The code is in a releasable state. A few questions: - Building a binary of a library in Go is not useful/meaningful. For the release, we just need to tar gz the git repo and sign it. Do you still need a script for this? Otherwise we can write the instructions somewhere in the

Re: Supporting named row construction in Calcite SQL

2018-04-16 Thread Julian Hyde
Beam and HerdDB are now on “powered by”: https://calcite.apache.org/docs/powered_by.html > On Apr 12, 2018, at 5:33 AM, Michael Mior wrote: > > FYI, I talked to Julian earlier this week and he will be adding Beam to the >