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 <a...@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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to