[GitHub] chetanmeh commented on issue #3515: Introduce separate Akka dispatchers for CouchDB and Kafka Clients (#2954)

2018-06-22 Thread GitBox
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)

2018-06-21 Thread GitBox
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)

2018-06-12 Thread GitBox
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