[GitHub] phoenix pull request: Fix for performance.py not using JAVA_HOME

2015-10-07 Thread bpanneton
Github user bpanneton commented on the pull request: https://github.com/apache/phoenix/pull/119#issuecomment-146227005 Updated to have sqlline, sqlline-thing and psql in there. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] phoenix pull request: Fix for performance.py not using JAVA_HOME

2015-10-02 Thread bpanneton
Github user bpanneton commented on the pull request: https://github.com/apache/phoenix/pull/119#issuecomment-145143153 Fixed the java that I missed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] phoenix pull request: Fix for performance.py not using JAVA_HOME

2015-10-02 Thread bpanneton
Github user bpanneton commented on the pull request: https://github.com/apache/phoenix/pull/119#issuecomment-145114015 Patch should now follow how queryserver.py gets the JAVA_HOME dir --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] phoenix pull request: Fix for performance.py not using JAVA_HOME

2015-10-02 Thread bpanneton
Github user bpanneton commented on the pull request: https://github.com/apache/phoenix/pull/119#issuecomment-145112736 Yea, that would be better. I'll fix it up. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] phoenix pull request: Fix for performance.py not using JAVA_HOME

2015-10-02 Thread bpanneton
GitHub user bpanneton opened a pull request: https://github.com/apache/phoenix/pull/119 Fix for performance.py not using JAVA_HOME You can merge this pull request into a Git repository by running: $ git pull https://github.com/bpanneton/phoenix missing-java-home-performance