markusthoemmes commented on issue #3283: Add implicit ClassTag for entity type
URL: 
https://github.com/apache/incubator-openwhisk/pull/3283#issuecomment-365511271
 
 
   Okay makes sense.
   
   This is a bit smelly though. It suggests we have a generalized API for 
something which shouldn't be generalized? Since we already have the 
`WhiskAuthStore`, `WhiskEntityStore`, `WhiskActivationStore` objects to 
construct entity stores, would it make sense to rather widen the constructor of 
`ArtifactStore`? What is the type of `WhiskHandler` for instance?

----------------------------------------------------------------
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

Reply via email to