[GitHub] markusthoemmes commented on a change in pull request #4219: Send capacity, system overload metrics for managed and blackbox invokers separately

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

markusthoemmes commented on a change in pull request #4219: Send capacity, system overload metrics for managed and blackbox invokers separately
URL: https://github.com/apache/incubator-openwhisk/pull/4219#discussion_r247902359
 
 

 ##########
 File path: common/scala/src/main/scala/org/apache/openwhisk/common/Logging.scala
 ##########
 @@ -330,14 +330,14 @@ object LoggingMarkers {
   val CONTROLLER_ACTIVATION_BLOCKING_DATABASE_RETRIEVAL =
     LogMarkerToken(controller, "blockingActivationDatabaseRetrieval", count)
 
-  // Time that is needed load balance the activation
+  // Time that is needed to load balance the activation
   val CONTROLLER_LOADBALANCER = LogMarkerToken(controller, loadbalancer, start)
 
   // Time that is needed to produce message in kafka
   val CONTROLLER_KAFKA = LogMarkerToken(controller, kafka, start)
 
   // System overload and random invoker assignment
-  val SYSTEM_OVERLOAD = LogMarkerToken(controller, "systemOverload", count)
+  def SYSTEM_OVERLOAD(actionType: String) = LogMarkerToken(controller, s"systemOverload$actionType", count)
 
 Review comment:
   Agreed, let's make two static values SYSTEM_OVERLOAD_MANAGED and SYSTEM_OVERLOAD_BLACKBOX to get rid of that work on each invocation.

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