prateekm commented on a change in pull request #986: SAMZA-2156: Couchbase
support for Samza Table API
URL: https://github.com/apache/samza/pull/986#discussion_r276337668
##########
File path: build.gradle
##########
@@ -697,6 +698,34 @@ project(":samza-kv-rocksdb_$scalaSuffix") {
}
}
+project(":samza-kv-remote_$scalaSuffix") {
Review comment:
Why? Is there something shared across all providers? If so, does it makes
sense to separate them into two packages - samza-kv-remote and
samza-kv-couchbase? If we add another remote table provider in the future, it
doesn't make sense to require users to pull in couchbase dependencies as well.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
