mhenke1 commented on a change in pull request #2990: Change log level 
dynamically
URL: 
https://github.com/apache/incubator-openwhisk/pull/2990#discussion_r154031014
 
 

 ##########
 File path: common/scala/src/main/scala/whisk/common/Logging.scala
 ##########
 @@ -37,7 +37,11 @@ trait Logging {
    * @param message Message to write to the log
    */
   def debug(from: AnyRef, message: String)(implicit id: TransactionId = 
TransactionId.unknown) = {
-    emit(DebugLevel, id, from, message)
+    if (id.meta.extraLogging) {
+      emit(InfoLevel, id, from, message)
 
 Review comment:
   @csantanapr They are not reversed. In the case of one transaction got marked 
with extra logging we elevate those messages from DEBUG to INFO. As I tried to 
explained in the comment above, this is the only way to get them placed into 
Akka logging's event bus when the overall log-level is set to INFO. 

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to