You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@samza.apache.org by prateekm <gi...@git.apache.org> on 2017/10/19 02:20:41 UTC

[GitHub] samza pull request #332: Minor fixes to KeyValueStore and RocksDBKeyValueSto...

GitHub user prateekm opened a pull request:

    https://github.com/apache/samza/pull/332

    Minor fixes to KeyValueStore and RocksDBKeyValueStore

    1. Replaced extension class in KeyValueStore with default methods.
    2. Fixed formatting in RocksDBKeyValueStore#openDB.
    3. Now logs original RocksDBException on errors opening the db. Other minor log message cleanup.

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

    $ git pull https://github.com/prateekm/samza store-fixes

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

    https://github.com/apache/samza/pull/332.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 #332
    
----
commit d727a4063388030d31238fe78c32ed522a63c6b3
Author: Prateek Maheshwari <pm...@linkedin.com>
Date:   2017-10-16T22:53:30Z

    Store fixes.

commit 7e6f3f1ab99c9e9a32b729359cb076a17855b4ff
Author: Prateek Maheshwari <pm...@linkedin.com>
Date:   2017-10-16T23:18:29Z

    Fixed formatting.

commit 09e0e87fbf214608536a4568324a1c0fc282e03f
Author: Prateek Maheshwari <pm...@linkedin.com>
Date:   2017-10-17T21:26:20Z

    Rethrow with original exception.

commit 9ff30e9c4c5fd19ab21f4b14c634672a3bd4a943
Author: Prateek Maheshwari <pm...@linkedin.com>
Date:   2017-10-19T02:17:23Z

    Revert unrelated changes

----


---

[GitHub] samza pull request #332: Minor fixes to KeyValueStore and RocksDBKeyValueSto...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/samza/pull/332


---