[GitHub] mdeuser commented on a change in pull request #3798: Allow for activation store to accept user and request information

[GitBox <gi...@apache.org>]

mdeuser commented on a change in pull request #3798: Allow for activation store to accept user and request information
URL: https://github.com/apache/incubator-openwhisk/pull/3798#discussion_r210410840
 
 

 ##########
 File path: common/scala/src/main/scala/whisk/core/containerpool/logging/DockerToActivationLogStore.scala
 ##########
 @@ -29,6 +28,7 @@ import whisk.common.TransactionId
 import whisk.core.containerpool.Container
 import whisk.core.entity.{ActivationLogs, ExecutableWhiskAction, Identity, WhiskActivation}
 import whisk.http.Messages
+import whisk.core.database.UserContext
 
 Review comment:
   is this class tightly coupled with the `database` package?  on the surface, it seems like it could go into some place else.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services