[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16122695#comment-16122695 ] Julian Hyde commented on CALCITE-1240: -- Fixed in http://git-wip-us.apache.org/repos/asf/calcite-avatica-go/commit/cfc09cd. Thank you for donating this to Apache, [~francischuang]! I have logged CALCITE-1937 (web site) and CALCITE-1938 (first release) as follow-ups. Thanks for your explanation of Gopkg.lock above. It makes sense that we include it in the source distribution. Likewise thanks for the explanation of the vendor directory. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica-go >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16122678#comment-16122678 ] ASF GitHub Bot commented on CALCITE-1240: - Github user asfgit closed the pull request at: https://github.com/apache/calcite-avatica-go/pull/1 > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16120990#comment-16120990 ] Francis Chuang commented on CALCITE-1240: - Hey [~julianhyde] Thanks for the comments. I added the header to wercker.yml, but I am not sure how to add the header to README.md and .gitignore. See the following in the calcite repo: https://github.com/apache/calcite/blob/master/README and https://github.com/apache/calcite/blob/master/.gitignore Gopkg.lock is a file generated by dep (https://github.com/golang/dep). It is poised to become the official package manager for Go. This file must be committed as it contains the exact commit hashes of the dependencies installed. The Gopkg.toml file only contains the constraints for example the minor version of a dependency that is versioned at 1.2.1. Therefore, the lock file is required for reproducible builds. It is equivalent to the lock file used by NPM 5. The vendor directory contains dependencies used by the calcite-avatica-go library. When running tests, libraries and projects to not run tests in the vendor directory. The vendored dependencies should have their own tests and the owner of the dependencies should be testing them. It does not make any sense to build the library. The library should be add into a project using dep, which will copy it to the project's vendor directory. On compilation, Go will build a single static binary for the project. I've pushed updated the PR with changes to the readme and a license header to wercker.yml. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16120925#comment-16120925 ] Julian Hyde commented on CALCITE-1240: -- To be completist, can you add headers to README.md, .gitignore, and wercker.yml (see how we do it in Calcite for README.md, .gitignore, and .travis.yml). I don't know much about Go; did you intend to check in Gopkg.lock? If it's generated there's a good chance that it shouldn't be. README.md has a tantalizing mention of moby.yml; the "we hope to open source it" pertains to Boostport not Apache Calcite project; remove it? Also, explain what the "vendor" directory is and why you'd want to skip tests in it. README.md doesn't seem to include instructions on how to build the software from source. (Not necessary now, but necessary before release.) > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16120894#comment-16120894 ] Francis Chuang commented on CALCITE-1240: - I've added the Apache License header to the files. I will probably not add the pre-Apache tags, as the original boostport/avatica repo will still be around for those using the old import path (there will be a message in the README directing people to the new repo). Here's the current todo list: * Create release history page * Switch CI to Apache Jenkins * Use self-hosted coverage reporting tool (maybe sonarqube's golang plugin) * Update URL in the awesome-go list * Update driver description on Avatica website > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16120734#comment-16120734 ] Josh Elser commented on CALCITE-1240: - bq. CI is important, but let's deal with it in a separate case. I think adding Apache headers is the only change necessary before this PR can go in. Agreed. bq. Jenkins uses a jenkinsfile which is stored in the project repo, but I do not see the jenkinsfile in the calcite or kafka repos. How are builds configured? You can do it this route if you'd like, but I don't have much experience. For the other Jenkins jobs for Calcite and Avatica, I've just created the jobs by hand. It is just an administrative task to get you an account. I believe there's at least one person in HBase who has experience doing this against their master branch. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16120619#comment-16120619 ] Julian Hyde commented on CALCITE-1240: -- CI is important, but let's deal with it in a separate case. I think adding Apache headers is the only change necessary before this PR can go in. I don't feel strongly one way or the other about tags. There's no harm in recognizing pre-Apache releases, both in tags and on the history page. But people won't be able to download pre-Apache releases from Apache, or even via an Apache page. Otherwise it would seem that Apache endorsed such releases, which it can't, because they haven't passed a PMC vote. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16119358#comment-16119358 ] Francis Chuang commented on CALCITE-1240: - I did some research regarding using Apache's infrastructure for CI and the ASF buildbot. I have some questions regarding how to set this up: - What needs to be done to set up CI for the project? - Can Apache's Jenkins build PRs submitted on github as well? - Jenkins uses a jenkinsfile which is stored in the project repo, but I do not see the jenkinsfile in the calcite or kafka repos. How are builds configured? There is a sonarqube instance: https://wiki.apache.org/general/SonarInstance Maybe it can provide code coverage, but I think the Go plugin needs to be installed: https://github.com/uartois/sonar-golang Ideally, there should be build status and coverage status images that can be embedded in the README.md file. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16119260#comment-16119260 ] Francis Chuang commented on CALCITE-1240: - [~elserj]Thanks for sorting that out! [~elserj][~julianhyde] A PR has been opened on the GH repo to rack the initial commit. There are still a few things to do before it can be merged. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[
https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16118436#comment-16118436
]
Josh Elser commented on CALCITE-1240:
-
[~francischuang], I pushed an initial commit that adds a basic LICENSE and
NOTICE file to the repository.
You should be able to create a fork from the apache/calcite-avatica-go
repository now. You can create a pull-request, but you would still need to
commit it via the canonical repository
https://git-wip-us.apache.org/repos/asf/calcite-avatica-go.git
Typically what I have is multiple remotes set up for a git repository:
* Origin points to the canonical ASF repository
* {{joshelser}} points to my GH fork of the repository on the {{apache}} org on
Github
So, you can {{git-fetch}} your changes from your personal GH, merge it into
{{master}} locally, and then push it to {{origin}} (canonical ASF repo).
> Avatica client written in Golang
>
>
> Key: CALCITE-1240
> URL: https://issues.apache.org/jira/browse/CALCITE-1240
> Project: Calcite
> Issue Type: Bug
> Components: avatica
>Reporter: Julian Hyde
>Assignee: Julian Hyde
>
> Add a client for Avatica written in the Go language (aka "Golang").
> There is one at https://github.com/Boostport/avatica and the author has
> offered to contribute it.
> The driver is currently somewhat specialized for Phoenix but our goal should
> be to allow it to work against any Avatica provider (without diminishing its
> value to Phoenix users).
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[
https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16118427#comment-16118427
]
Josh Elser commented on CALCITE-1240:
-
{quote}
I currently use Wercker for CI. Can wercker be used on Apache repos?
I use goveralls for code coverage reports. Can it be used on Apache repos?
{quote}
Ideally, it would be better to not rely on external systems, but I don't
believe there is a reason we cannot use them.
It would be good to transition to the ASF Jenkins server(s) for CI so that all
members of Calcite can improve it.
For code-coverage, using this as a tool is fine, IMO. There are some tools
which may be available at the ASF already, but I'm not sure how they compare
WRT features. Is there a non-hosted options that developers could run on their
own?
> Avatica client written in Golang
>
>
> Key: CALCITE-1240
> URL: https://issues.apache.org/jira/browse/CALCITE-1240
> Project: Calcite
> Issue Type: Bug
> Components: avatica
>Reporter: Julian Hyde
>Assignee: Julian Hyde
>
> Add a client for Avatica written in the Go language (aka "Golang").
> There is one at https://github.com/Boostport/avatica and the author has
> offered to contribute it.
> The driver is currently somewhat specialized for Phoenix but our goal should
> be to allow it to work against any Avatica provider (without diminishing its
> value to Phoenix users).
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16118420#comment-16118420 ] Josh Elser commented on CALCITE-1240: - bq. Ok, the repo is now on Github, but I ran into a snag as I am not allowed to push directly to apache/calcite-avatica-go on github. Correct. The canonical repository is hosted on git-wip-us.apache.org. The Github "version" is just a copy/mirror for convenience. bq. it won't let me open a PR against apache/calcite-avatica-go. That does seem like a problem. Let me see what I can do. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16118054#comment-16118054 ] Francis Chuang commented on CALCITE-1240: - Ok, the repo is now on Github, but I ran into a snag as I am not allowed to push directly to apache/calcite-avatica-go. - The repo is currently empty, so I cannot fork it on GH and use a fork. - I created a F21/calcite-avatica-go repo and pushed the initial commit there, but it won't let me open a PR against apache/calcite-avatica-go. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16117182#comment-16117182 ] Josh Elser commented on CALCITE-1240: - The canonical ASF repo is live at https://git-wip-us.apache.org/repos/asf?p=calcite-avatica-go.git The Github mirror is still in the process of being created, but should show up [here|https://github.com/apache?utf8=%E2%9C%93&q=calcite&type=&language=] eventually > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16116835#comment-16116835 ] Julian Hyde commented on CALCITE-1240: -- Sorry, I didn't actually check. I have limited network bandwidth. Josh, Please create it. Francis, Up to you whether you push the history or just the latest. Either is fine. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16116813#comment-16116813 ] Josh Elser commented on CALCITE-1240: - I'm not seeing it either. Maybe INFRA just meant that they fixed the reporeq tool and didn't actually create the repo for you, [~julianhyde]? I can create it too -- I just don't want to cause a race-condition between us :) > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16116311#comment-16116311 ] Francis Chuang commented on CALCITE-1240: - Also, I can't find the avatica-go repo in the Apache org on github. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16116173#comment-16116173 ] Francis Chuang commented on CALCITE-1240: - [~julianhyde]Should I push the whole commit history from the old repo, or just the latest commit (without the history)? > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16116085#comment-16116085 ] Julian Hyde commented on CALCITE-1240: -- [~francischuang], I see that INFRA-14777 is marked fixed, so there is now a repo. Would you mind creating a pull request for the initial version of the repo? Josh and I will review. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16111771#comment-16111771 ] Julian Hyde commented on CALCITE-1240: -- The vote on general@incubator ([thread|https://s.apache.org/calcite-avatica-go-clearance-vote]) has passed ([result|https://s.apache.org/calcite-avatica-go-clearance-result]). > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16110005#comment-16110005 ] Francis Chuang commented on CALCITE-1240: - [~julianhyde] Thanks for organizing this! An empty repo is fine, I can push the existing code into the repo. Should I push the existing tags or should we only have the tags for when the code is part of the Apache foundation? I agree that the headers docs and other pieces will need to be updated. Is there an official list of things we should do before tagging the first release? > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16109983#comment-16109983 ] Julian Hyde commented on CALCITE-1240: -- Vote is in progress now. [~francischuang], When the vote finishes, what are the next steps? Would you like an empty git repo (into which you can load the latest source) or a repo that is a copy of https://github.com/boostport/avatica? Then I think we should make sure files have Apache headers, update documentation, create a release history page, and create a release candidate, update Avatica's web site. What do you think? > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[
https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16105381#comment-16105381
]
Josh Elser commented on CALCITE-1240:
-
bq. AFAICT, /repos/asf/incubator/donations doesn't exist. Is this a typo in the
template? What did you do when you handled previous IP clearance?
Hrm, that's interesting. I don't think I committed the granted codebase to SVN
before. Pulling up the old thread, I just hosted it in my ~elserj space on
people.a.o ("https://github.com/apache/incubator-slider/pull/3 or
http://people.apache.org/~elserj/KOYA-grant.patch"; is specifically what I put
in the IP clearance XML form). I would say what you've done is sufficient --
someone can correct us during the lazy-consensus period if that is incorrect.
> Avatica client written in Golang
>
>
> Key: CALCITE-1240
> URL: https://issues.apache.org/jira/browse/CALCITE-1240
> Project: Calcite
> Issue Type: Bug
> Components: avatica
>Reporter: Julian Hyde
>Assignee: Julian Hyde
>
> Add a client for Avatica written in the Go language (aka "Golang").
> There is one at https://github.com/Boostport/avatica and the author has
> offered to contribute it.
> The driver is currently somewhat specialized for Phoenix but our goal should
> be to allow it to work against any Avatica provider (without diminishing its
> value to Phoenix users).
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16105362#comment-16105362 ] Julian Hyde commented on CALCITE-1240: -- [~elserj], I got stuck on one step: bq. If the source is referenced by checksum in the grant, commit the canonical tarball for the donated code into the incubator drop area (/repos/asf/incubator/donations) together with a checksum and a detached signature. This will ensure that apache has a legal record of the grant. AFAICT, /repos/asf/incubator/donations doesn't exist. Is this a typo in the template? What did you do when you handled previous IP clearance? As mentioned above, I put the patch in my home directory. I don't know whether that is sufficient. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16105345#comment-16105345 ] Julian Hyde commented on CALCITE-1240: -- The Boostport CCLA will do fine. (It is present and correct at https://svn.apache.org/repos/private/foundation/officers/cclas.txt.) I have filled out and committed the form: http://incubator.apache.org/ip-clearance/calcite-avatica-go.html It includes a patch http://people.apache.org/~jhyde/boostport-avatica.patch based on https://github.com/Boostport/avatica/commit/77207918cf826662cb4ea40cfffbfb5cb64bf4a0. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16104348#comment-16104348 ] Francis Chuang commented on CALCITE-1240: - Hey [~julianhyde] Thanks for getting the ball rolling. A corporate CLA for Boostport Pty Ltd was signed in September last year when I was accepted as a committer. An acknowledgement was CC'd to the calcite private list on the 6th of September, 2016. Is this sufficient? If not, I'll fill in the Software Grant Agreement and send that to secret...@apache.org Using the current HEAD should be fine. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16104286#comment-16104286 ] Julian Hyde commented on CALCITE-1240: -- OK, starting this process now. [~francischuang], can you please complete step 3 of the [IP clearance template|http://incubator.apache.org/ip-clearance/ip-clearance-template.html]: bq. 3. A software grant must be provided to the ASF. This grant can either be done by the ASF Corporate CLA (via Schedule B) or the Software Grant Agreement. The completed and signed grant must be emailed to secret...@apache.org I suggest that we use the current HEAD, https://github.com/Boostport/avatica/commit/77207918cf826662cb4ea40cfffbfb5cb64bf4a0, the code donation. If you agree, I will create a tar ball and commit it to the incubator drop area. I am currently filling out the form and plan to commit it as http://svn.apache.org/repos/asf/incubator/public/trunk/content/ip-clearance/calcite-avatica-boostport-go.xml. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16095599#comment-16095599 ] Julian Hyde commented on CALCITE-1240: -- Shouldn't be necessary. Category b licensed dependencies are OK, as long as we're careful. We can deal with this the first release after the code is accepted. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16095590#comment-16095590 ] Francis Chuang commented on CALCITE-1240: - go-cleanhttp is a really simple library. All it does is set up the http client with sane defaults: https://github.com/hashicorp/go-cleanhttp/blob/master/cleanhttp.go If required, I can remove the dependency on it and initialize the http client manually with those defaults. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16095559#comment-16095559 ] Julian Hyde commented on CALCITE-1240: -- It seems that [go-cleanhttp|https://github.com/hashicorp/go-cleanhttp/blob/master/LICENSE] is MPL, which is [category b (weak copyleft)|https://www.apache.org/legal/resolved.html#category-b]. We'll have to be careful with that. The others are Apache, BSD or MIT, so they're OK. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16095516#comment-16095516 ] Francis Chuang commented on CALCITE-1240: - For the dependencies: Other than Go's standard library, the project also imports the following dependencies directly: https://github.com/Boostport/avatica/blob/master/Gopkg.toml The imported dependencies also imports further dependencies of their own. The list of ALL dependencies used are in here: https://github.com/Boostport/avatica/blob/master/Gopkg.lock Dependencies under the golang.org/x/ namespace are part of the Go project but are considered to be experimental. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16095179#comment-16095179 ] Julian Hyde commented on CALCITE-1240: -- Actually, step 0 would be a *discussion* on dev. Let me start that. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16094934#comment-16094934 ] Julian Hyde commented on CALCITE-1240: -- Here are the steps to go through the [IP clearance process|http://incubator.apache.org/ip-clearance/]. The steps are: # Vote on dev about accepting the code into the project # Add new IP clearance proposal to http://incubator.apache.org/ip-clearance/ # Contributor(s) (i.e. [~francischuang]) execute a [software grant document|https://www.apache.org/licenses/software-grant.txt] and PMC sends to the ASF secretary # Check that contributor(s) has filed an ICLA with ASF # Once the Software Grant has been acknowledged by the ASF secretary and the ICLAs are on file, start a lazy consensus vote about the IP clearance on general@incubator A separate git repo is certainly possible. I can see the benefits. I have a small concern that it might make unified releases of Avatica-Java and Avatica-Go more difficult (if we ever decided to do that). [~julienledem] since you've been through this, do you have any regrets about splitting Parquet into several repositories? > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16094799#comment-16094799 ] Josh Elser commented on CALCITE-1240: - bq. 2. Is it possible to get a separate git repo for this? It makes it easier to import using Go's dependency tools like dep and the package structure would be more idiomatic. Yup bq. What is the process for getting the code into the Calcite/Avatica project? Essentially, we'll have to follow the IP Clearance process. https://incubator.apache.org/ip-clearance/index.html This isn't anything more than some formality in documenting: license compatibility, copyright assignment, and such. Working through the Go dependencies to make sure we follow ASF policies will be the biggest issue :) > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (CALCITE-1240) Avatica client written in Golang
[ https://issues.apache.org/jira/browse/CALCITE-1240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16093971#comment-16093971 ] Francis Chuang commented on CALCITE-1240: - [~julianhyde] A few questions: 1. What is the process for getting the code into the Calcite/Avatica project? 2. Is it possible to get a separate git repo for this? It makes it easier to import using Go's dependency tools like dep and the package structure would be more idiomatic. > Avatica client written in Golang > > > Key: CALCITE-1240 > URL: https://issues.apache.org/jira/browse/CALCITE-1240 > Project: Calcite > Issue Type: Bug > Components: avatica >Reporter: Julian Hyde >Assignee: Julian Hyde > > Add a client for Avatica written in the Go language (aka "Golang"). > There is one at https://github.com/Boostport/avatica and the author has > offered to contribute it. > The driver is currently somewhat specialized for Phoenix but our goal should > be to allow it to work against any Avatica provider (without diminishing its > value to Phoenix users). -- This message was sent by Atlassian JIRA (v6.4.14#64029)
