tephra-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From anew <...@git.apache.org>
Subject [GitHub] incubator-tephra pull request #47: [TEPHRA-240] Include conflicting key and ...
Date Thu, 17 Aug 2017 23:28:09 GMT
GitHub user anew opened a pull request:

    https://github.com/apache/incubator-tephra/pull/47

    [TEPHRA-240] Include conflicting key and client id in TransactionConflictException

    - Enhances TransactionConflictException to have fields for the transaction id, the conflicting
change, and the conflicting client id
    - TransactionManager to retain the client id for change sets after the transaction has
committed
    - A configuration for whether and how long it should be retained. This is to allow disabling
this feature (and it effect on memory)
    - TransactionManager.(can)Commit now throws TrandactionConflictException instead of returning
a boolean
    - new thrift methods (and a new thrift exception). The existing thrift methods will remain
backward-compatible
    - change TransactionSystemClient and TransactionContext to use the new methods
    - new test case (and existing test case changes) for this
    - a few unrelated warnings have been removed from TransactionManager
    - a bit of test case refactoring to remove duplication of code

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/anew/incubator-tephra tephra-240

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-tephra/pull/47.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 #47
    
----
commit 205714fdd126ab7537f9f66e6e156c9736f54646
Author: anew <anew@apache.org>
Date:   2017-08-17T23:21:52Z

    [TEPHRA-240] Include conflicting key and client id in TransactionConflictException

----


---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message