[GitHub] chetanmeh commented on issue #3515: Introduce separate Akka dispatchers for CouchDB and Kafka Clients (#2954)
chetanmeh commented on issue #3515: Introduce separate Akka dispatchers for CouchDB and Kafka Clients (#2954) URL: https://github.com/apache/incubator-openwhisk/pull/3515#issuecomment-399368110 @bwmcadams There are 2 aspects 1. Have a dedicated separate dispatcher for db operations 2. Dispatcher being tuned per DB driver implementation Currently we have few storage impls which use Akka HTTP stack like CouchDB, S3 and ElasticSearch. So for them I think it would be better to have a generic `db-dispatcher` which is used by default. If any setup needs specific tuning then config can still be overridden for that deployment and later we can see if it makes sense to use a dedicated dispatcher for that kind of db. So to start with lets have a generic `db-dispatcher` and later based on learning we can define specific one. This would allow us to at least meet 1 and still enabling option for 2. Thoughts? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] chetanmeh commented on issue #3515: Introduce separate Akka dispatchers for CouchDB and Kafka Clients (#2954)
chetanmeh commented on issue #3515: Introduce separate Akka dispatchers for CouchDB and Kafka Clients (#2954) URL: https://github.com/apache/incubator-openwhisk/pull/3515#issuecomment-399043207 @bwmcadams #3779 introduces a new `S3AttachmentStore` where I was thinking to use the dedicated dispatcher. Do you plan to rename the dispatcher from `couch-dispatcher` to `db-dispatcher`? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] chetanmeh commented on issue #3515: Introduce separate Akka dispatchers for CouchDB and Kafka Clients (#2954)
chetanmeh commented on issue #3515: Introduce separate Akka dispatchers for CouchDB and Kafka Clients (#2954) URL: https://github.com/apache/incubator-openwhisk/pull/3515#issuecomment-396540431 > this might be of use for other datastores so I wonder if we should rename the couch configuration to data store. +1 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services