You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by "ijuma (via GitHub)" <gi...@apache.org> on 2023/07/03 20:12:53 UTC

[GitHub] [kafka] ijuma commented on a diff in pull request #13949: KAFKA-15141: init logger statically on hot codepaths

ijuma commented on code in PR #13949:
URL: https://github.com/apache/kafka/pull/13949#discussion_r1251251454


##########
core/src/main/scala/kafka/server/DelayedProduce.scala:
##########
@@ -58,6 +62,8 @@ class DelayedProduce(delayMs: Long,
                      lockOpt: Option[Lock] = None)
   extends DelayedOperation(delayMs, lockOpt) {
 
+  override lazy val logger = DelayedProduce.logger

Review Comment:
   Instead of doing this, you can simply import the logger from the companion object.



-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org