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/05 02:00:30 UTC

[GitHub] samza pull request #315: Removed circular dependency b/w samza-core test and...

GitHub user prateekm opened a pull request:

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

    Removed circular dependency b/w samza-core test and samza-kv-rocksdb test

    Also added an implementation for KVSerde.
    @vjagadish1989, thanks for the TestInMemoryStore implementation!
    
    The only usage of InternalInMemoryStore is now in WindowOperatorImpl, which will be removed as part of store wiring for the window operator.

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

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

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

    https://github.com/apache/samza/pull/315.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 #315
    
----
commit e2a4233e704725865081dd62133cc443484c3641
Author: Prateek Maheshwari <pm...@linkedin.com>
Date:   2017-10-05T01:57:48Z

    Removed circular dependency b/w samza-core test and samza-kv-rocksdb test

----


---

[GitHub] samza pull request #315: SAMZA-1444: Removed circular dependency b/w samza-c...

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

Github user asfgit closed the pull request at:

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


---