[
https://issues.apache.org/jira/browse/PHOENIX-900?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14323312#comment-14323312
]
ASF GitHub Bot commented on PHOENIX-900:
----------------------------------------
GitHub user elilevine opened a pull request:
https://github.com/apache/phoenix/pull/37
Surface partial saves in CommitExcepiton (PHOENIX-900)
https://issues.apache.org/jira/browse/PHOENIX-900
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/elilevine/apache-phoenix PHOENIX-900
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/phoenix/pull/37.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #37
----
commit 10754f7ab71d62349acb2e18e79a91ded7bc6c62
Author: Eli Levine <[email protected]>
Date: 2015-02-06T00:28:33Z
First pass at impl. Not tested.
commit d8ea1db808434ca1d159d6316105d50dbd0bca82
Author: Eli Levine <[email protected]>
Date: 2015-02-10T01:40:28Z
First attempt at a new integ test for partial commits using a coproc. Still
WIP.
commit 34302cfac39d3d893103a99886d10a3fc8e65d58
Author: Eli Levine <[email protected]>
Date: 2015-02-11T23:16:28Z
Test now successfully verifies partial upsert failure. Statement ordering
still borked. Prolly need to swith to keeping order of statement execution
instead of creation.
commit 6f7bc57646543a08222e652f264ee02b6b3e1241
Author: Eli Levine <[email protected]>
Date: 2015-02-12T00:41:34Z
Switched to keeping a count of statement executions instead of creations,
which is ultimately used in MutationState. Basic partial commit test working.
commit 74e3ba6ce1bf12de62d1b3859e23b6ce224bd370
Author: Eli Levine <[email protected]>
Date: 2015-02-12T02:04:14Z
Modify PartialCommitIT to add select clause
commit 265b2e90056bca13b8ba59efba15841c9a8658ef
Author: Eli Levine <[email protected]>
Date: 2015-02-13T01:26:51Z
Separated getting and incrementing of PhoenixConnection's statement
execution counter. More tests. Still WIP.
commit 0f8958b3debb2f391a230f51b29fe102b426c9be
Author: Eli Levine <[email protected]>
Date: 2015-02-15T22:55:21Z
Fix partial success w/ delete test
commit 29ae22f5c3bec7dbd659b16edcaa4904fd1c1cd8
Author: Eli Levine <[email protected]>
Date: 2015-02-16T04:43:03Z
Clarify that partial save statement tracking is only supported for
operations on data, not metadata.
commit 5ee273cf36729fab0bf9613bfead5933c920bd35
Author: Eli Levine <[email protected]>
Date: 2015-02-16T19:55:30Z
Add data verification after partial saves to PartialCommitIT
----
> Partial results for mutations
> -----------------------------
>
> Key: PHOENIX-900
> URL: https://issues.apache.org/jira/browse/PHOENIX-900
> Project: Phoenix
> Issue Type: Bug
> Affects Versions: 3.0.0, 4.0.0
> Reporter: Eli Levine
> Assignee: Eli Levine
> Attachments: PHOENIX-900.patch
>
>
> HBase provides a way to retrieve partial results of a batch operation:
> http://hbase.apache.org/apidocs/org/apache/hadoop/hbase/client/HTable.html#batch%28java.util.List,%20java.lang.Object[]%29
> Chatted with James about this offline:
> Yes, this could be included in the CommitException we throw
> (MutationState:412). We already include the batches that have been
> successfully committed to the HBase server in this exception. Would you be up
> for adding this additional information? You'd want to surface this in a
> Phoenix-y way in a method on CommitException, something like this: ResultSet
> getPartialCommits(). You can easily create an in memory ResultSet using
> MaterializedResultIterator plus the PhoenixResultSet constructor that accepts
> this (just create a new empty PhoenixStatement with the PhoenixConnection for
> the other arg).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
