gianm commented on PR #13138:
URL: https://github.com/apache/druid/pull/13138#issuecomment-1255339885

   > Went ahead and added a log line containing the `ExtensionsConfig` in the 
constructor of the task post the approvals.
   
   Please don't add it there: tasks are serialized and deserialized at various 
times where they aren't actually run, so we don't want the constructor to log 
anything. Log messages should go to the `run` method.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to