phoenix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (PHOENIX-900) Partial results for mutations
Date Mon, 16 Feb 2015 21:21:11 GMT

    [ 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 <elilevine@apache.org>
Date:   2015-02-06T00:28:33Z

    First pass at impl. Not tested.

commit d8ea1db808434ca1d159d6316105d50dbd0bca82
Author: Eli Levine <elilevine@apache.org>
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 <elilevine@apache.org>
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 <elilevine@apache.org>
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 <elilevine@apache.org>
Date:   2015-02-12T02:04:14Z

    Modify PartialCommitIT to add select clause

commit 265b2e90056bca13b8ba59efba15841c9a8658ef
Author: Eli Levine <elilevine@apache.org>
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 <elilevine@apache.org>
Date:   2015-02-15T22:55:21Z

    Fix partial success w/ delete test

commit 29ae22f5c3bec7dbd659b16edcaa4904fd1c1cd8
Author: Eli Levine <elilevine@apache.org>
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 <elilevine@apache.org>
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)

Mime
View raw message