Github user apurtell commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/292#discussion_r166479944
--- Diff:
phoenix-core/src/main/java/org/apache/phoenix/jdbc/PhoenixStatement.java ---
@@ -907,10 +909,15 @@ public MutationState execute() throws
Github user apurtell commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/292#discussion_r166479267
--- Diff:
phoenix-core/src/main/java/org/apache/phoenix/jdbc/PhoenixStatement.java ---
@@ -907,10 +909,15 @@ public MutationState execute() throws
Github user apurtell commented on the issue:
https://github.com/apache/phoenix/pull/240
@JamesRTaylor No I haven't been working on this. Maybe next week?
---
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 pr
Github user apurtell commented on the issue:
https://github.com/apache/phoenix/pull/236
Also please resolve indicated conflicts.
---
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 not have this feature
Github user apurtell commented on the issue:
https://github.com/apache/phoenix/pull/236
@rahulsIOT Please rename this issue "PHOENIX-3654 Load Balancer for thin
client" and please squash all commits and set the commit message to
"PHOENIX-3654 Load Balancer for thi
Github user apurtell commented on the issue:
https://github.com/apache/phoenix/pull/245
Any chance of a review on #240 ?
---
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 not have this feature
enabled
GitHub user apurtell opened a pull request:
https://github.com/apache/phoenix/pull/240
PHOENIX-3808 Implement chaos tests using HBase's hbase-it facility
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apurtell/phoenix PH
Github user apurtell commented on the issue:
https://github.com/apache/phoenix/pull/230
We committed HBASE-17587
---
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 not have this feature
enabled and
Github user apurtell commented on the issue:
https://github.com/apache/phoenix/pull/230
Thanks @z-york . I plan to commit HBASE-17587 if tests pass there
---
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
Github user apurtell commented on the issue:
https://github.com/apache/phoenix/pull/230
Phoenix development is hosted at the Apache foundation and primarily uses
JIRA at https://issues.apache.org/jira/browse/PHOENIX for issue tracking and
the mailing list dev@phoenix.apache.org (see
Github user apurtell commented on the issue:
https://github.com/apache/phoenix/pull/230
lgtm
Have you tried running the Phoenix test suite with these changes in place
@z-york ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user apurtell commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/183#discussion_r71762055
--- Diff: dev/make_rc.sh ---
@@ -88,15 +88,15 @@ cp $DIR_DOCS/* $DIR_REL_BIN_PATH;
cp -r examples/* $DIR_EXAMPLES
# Generate bin tar
Github user apurtell commented on the issue:
https://github.com/apache/phoenix/pull/183
lgtm
---
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 not have this feature
enabled and wishes so, or if the
Github user apurtell commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/183#discussion_r71761538
--- Diff: dev/make_rc.sh ---
@@ -30,10 +30,10 @@ DIR_ROOT="$(cd $(dirname $0);pwd)/.."
cd $DIR_ROOT
PHOENIX="$(xmllint --xpath
Github user apurtell commented on the pull request:
https://github.com/apache/phoenix/pull/59#issuecomment-88248990
I'm not a Spark expert @JamesRTaylor . I skimmed the latest. Allowing
builds with JDK 1.7 would have been the big change I'd have recommended, and
it'
15 matches
Mail list logo
