[GitHub] phoenix pull request #291: [PHOENIX-4278] Implement pure client side transac...

2018-01-29 Thread ohadshacham
GitHub user ohadshacham opened a pull request: https://github.com/apache/phoenix/pull/291 [PHOENIX-4278] Implement pure client side transactional index mainten… …ance. Moving the index update from a coprocessor to the client side. This is done only for transactions

[GitHub] phoenix pull request #291: [PHOENIX-4278] Implement pure client side transac...

2018-02-08 Thread ohadshacham
Github user ohadshacham commented on a diff in the pull request: https://github.com/apache/phoenix/pull/291#discussion_r166868712 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/schema/PTableImpl.java --- @@ -1038,32 +1038,21 @@ public void setValue(PColumn column, byte

[GitHub] phoenix pull request #291: [PHOENIX-4278] Implement pure client side transac...

2018-02-08 Thread ohadshacham
Github user ohadshacham commented on a diff in the pull request: https://github.com/apache/phoenix/pull/291#discussion_r166875958 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java --- @@ -850,19 +849,12 @@ private void addCoprocessors

[GitHub] phoenix pull request #291: [PHOENIX-4278] Implement pure client side transac...

2018-02-08 Thread ohadshacham
Github user ohadshacham commented on a diff in the pull request: https://github.com/apache/phoenix/pull/291#discussion_r166876845 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/PhoenixTxnIndexMutationGenerator.java --- @@ -0,0 +1,505 @@ +/* + * Licensed

[GitHub] phoenix pull request #291: [PHOENIX-4278] Implement pure client side transac...

2018-02-08 Thread ohadshacham
Github user ohadshacham commented on a diff in the pull request: https://github.com/apache/phoenix/pull/291#discussion_r166908718 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/PhoenixTxnIndexMutationGenerator.java --- @@ -0,0 +1,505 @@ +/* + * Licensed

[GitHub] phoenix issue #291: [PHOENIX-4278] Implement pure client side transactional ...

2018-02-12 Thread ohadshacham
Github user ohadshacham commented on the issue: https://github.com/apache/phoenix/pull/291 @JamesRTaylor, anything else should be done before commit? ---

[GitHub] phoenix pull request #291: [PHOENIX-4278] Implement pure client side transac...

2018-02-12 Thread ohadshacham
Github user ohadshacham commented on a diff in the pull request: https://github.com/apache/phoenix/pull/291#discussion_r167501024 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java --- @@ -850,19 +849,12 @@ private void addCoprocessors

[GitHub] phoenix pull request #246: Connect tal to phoenix

2017-05-09 Thread ohadshacham
GitHub user ohadshacham opened a pull request: https://github.com/apache/phoenix/pull/246 Connect tal to phoenix You can merge this pull request into a Git repository by running: $ git pull https://github.com/ohadshacham/phoenix ConnectTALtoPhoenix Alternatively you can

[GitHub] phoenix pull request #246: Connect tal to phoenix

2017-05-09 Thread ohadshacham
Github user ohadshacham closed the pull request at: https://github.com/apache/phoenix/pull/246 --- 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 feature

[GitHub] phoenix pull request #246: Connect tal to phoenix

2017-05-09 Thread ohadshacham
GitHub user ohadshacham reopened a pull request: https://github.com/apache/phoenix/pull/246 Connect tal to phoenix You can merge this pull request into a Git repository by running: $ git pull https://github.com/ohadshacham/phoenix ConnectTALtoPhoenix Alternatively you can

[GitHub] phoenix pull request #246: [PHOENIX-3734] Refactor Phoenix to use TAL instea...

2017-11-29 Thread ohadshacham
Github user ohadshacham closed the pull request at: https://github.com/apache/phoenix/pull/246 ---