[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-04 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r109779896 --- Diff: docker/src/main/docker/Dockerfile --- @@ -0,0 +1,33 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-04 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r109780099 --- Diff: docker/src/main/dockerhub/Dockerfile --- @@ -0,0 +1,36 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-04 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r109781300 --- Diff: docker/src/main/docker/Dockerfile --- @@ -0,0 +1,33 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110018572 --- Diff: standalone-server/src/main/java/org/apache/calcite/avatica/standalone/StandaloneServer.java --- @@ -92,7 +96,12 @@ public void join

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110020200 --- Diff: docker/src/main/docker/postgresql/docker-compose.yml --- @@ -0,0 +1,35 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110019931 --- Diff: docker/src/main/docker/postgresql/docker-compose.yml --- @@ -0,0 +1,35 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110019830 --- Diff: docker/src/main/docker/mysql/docker-compose.yml --- @@ -0,0 +1,35 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110020852 --- Diff: docker/src/main/docker/postgresql/Dockerfile --- @@ -0,0 +1,24 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110021022 --- Diff: docker/src/main/docker/mysql/Dockerfile --- @@ -0,0 +1,24 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110020308 --- Diff: docker/src/main/docker/mysql/docker-compose.yml --- @@ -0,0 +1,35 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110021778 --- Diff: site/_docs/docker_images.md --- @@ -0,0 +1,125 @@ +--- +layout: docs +title: Docker Images +sidebar_title: Docker Images

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110022358 --- Diff: docker/src/main/docker/hypersql/Dockerfile --- @@ -0,0 +1,23 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110018738 --- Diff: standalone-server/pom.xml --- @@ -23,8 +23,8 @@ limitations under the License. 1.10.0-SNAPSHOT avatica-standalone

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110019153 --- Diff: docker/pom.xml --- @@ -0,0 +1,125 @@ + + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110020947 --- Diff: docker/src/main/docker/postgresql/Dockerfile --- @@ -0,0 +1,24 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110022130 --- Diff: docker/src/main/docker/Dockerfile --- @@ -0,0 +1,33 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110019480 --- Diff: docker/README.md --- @@ -0,0 +1,46 @@ + + +This module contains a number of Docker images to ease testing of +Avatica

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110021257 --- Diff: docker/README.md --- @@ -0,0 +1,46 @@ + + +This module contains a number of Docker images to ease testing of +Avatica

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110019665 --- Diff: docker/README.md --- @@ -0,0 +1,46 @@ + + +This module contains a number of Docker images to ease testing of +Avatica

[GitHub] calcite-avatica pull request #3: CALCITE-1364 Docker images for an avatica s...

2017-04-05 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/3#discussion_r110023201 --- Diff: standalone-server/src/main/java/org/apache/calcite/avatica/standalone/StandaloneServer.java --- @@ -92,7 +96,12 @@ public void join

[GitHub] calcite-avatica-go pull request #11: Add .travis.yml

2018-04-15 Thread risdenk
GitHub user risdenk opened a pull request: https://github.com/apache/calcite-avatica-go/pull/11 Add .travis.yml You can merge this pull request into a Git repository by running: $ git pull https://github.com/risdenk/calcite-avatica-go testtravis Alternatively you can review

[GitHub] calcite-avatica-go issue #11: Add .travis.yml

2018-04-15 Thread risdenk
Github user risdenk commented on the issue: https://github.com/apache/calcite-avatica-go/pull/11 Closing since this was a test. ---

[GitHub] calcite-avatica-go pull request #11: Add .travis.yml

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

[GitHub] calcite-avatica-go pull request #12: [CALCITE-2258] Add .travis.yml

2018-04-15 Thread risdenk
GitHub user risdenk opened a pull request: https://github.com/apache/calcite-avatica-go/pull/12 [CALCITE-2258] Add .travis.yml Addes .travis.yml based on existing wercker.yml. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] calcite-avatica-go pull request #12: [CALCITE-2258] Add .travis.yml

2018-04-15 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica-go/pull/12#discussion_r181610048 --- Diff: .travis.yml --- @@ -0,0 +1,54 @@ +# Configuration file for Travis continuous integration. +# See https://travis-ci.org/apache

[GitHub] calcite-avatica-go pull request #12: [CALCITE-2258] Add .travis.yml

2018-04-15 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica-go/pull/12#discussion_r181610032 --- Diff: .travis.yml --- @@ -0,0 +1,54 @@ +# Configuration file for Travis continuous integration. +# See https://travis-ci.org/apache

[GitHub] calcite-avatica-go issue #13: Add HSQLDB support and move phoenix support in...

2018-04-15 Thread risdenk
Github user risdenk commented on the issue: https://github.com/apache/calcite-avatica-go/pull/13 For reference I tried this against the new .travis.yml in my fork and got a build error: https://travis-ci.org/risdenk/calcite-avatica-go/builds/366966818 ``` Solving

[GitHub] calcite-avatica-go issue #13: Add HSQLDB support and move phoenix support in...

2018-04-15 Thread risdenk
Github user risdenk commented on the issue: https://github.com/apache/calcite-avatica-go/pull/13 Looks like this might have been a false alarm. ``` mkdir -p $HOME/gopath/src/github.com/risdenk/calcite-avatica-go ``` Travis was picking up my username for the gopath

[GitHub] calcite-avatica-go pull request #13: Add HSQLDB support and move phoenix sup...

2018-04-15 Thread risdenk
Github user risdenk commented on a diff in the pull request: https://github.com/apache/calcite-avatica-go/pull/13#discussion_r181618280 --- Diff: phoenix/phoenix.go --- @@ -1,82 +1,105 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one or more

[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 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_r181917580 --- Diff: make-release-artifacts.sh --- @@ -0,0 +1,26 @@ +# Clean dist directory +rm -rf dist +mkdir -p dist + +# Get new tags