You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "scholzj (via GitHub)" <gi...@apache.org> on 2023/02/07 19:34:57 UTC
[GitHub] [kafka] scholzj opened a new pull request, #13216: Remove unused ZooKeeper log level configuration from `connect-log4j.properties`
scholzj opened a new pull request, #13216:
URL: https://github.com/apache/kafka/pull/13216
The `connect-log4j.properties` file configures the log level for the ZooKeeper logger. That seems unnecessary since Kafka Connect does not use ZooKeeper directly in any way. This PR removes it from the logging configuration file.
--
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
[GitHub] [kafka] mimaison commented on pull request #13216: Remove unused ZooKeeper log level configuration from `connect-log4j.properties`
Posted by "mimaison (via GitHub)" <gi...@apache.org>.
mimaison commented on PR #13216:
URL: https://github.com/apache/kafka/pull/13216#issuecomment-1438720784
Thanks @scholzj for the PR. As suggested by Federico, can you also update `tests/kafkatest/services/templates/connect_log4j.properties`?
--
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
[GitHub] [kafka] mimaison merged pull request #13216: Remove unused ZooKeeper log level configuration from `connect-log4j.properties`
Posted by "mimaison (via GitHub)" <gi...@apache.org>.
mimaison merged PR #13216:
URL: https://github.com/apache/kafka/pull/13216
--
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
[GitHub] [kafka] fvaleri commented on pull request #13216: Remove unused ZooKeeper log level configuration from `connect-log4j.properties`
Posted by "fvaleri (via GitHub)" <gi...@apache.org>.
fvaleri commented on PR #13216:
URL: https://github.com/apache/kafka/pull/13216#issuecomment-1422141061
Hi @scholzj, thanks. I think this is not needed anymore, but we should probably keep the system tests aligned: `tests/kafkatest/services/templates/connect_log4j.properties`.
--
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
[GitHub] [kafka] scholzj commented on pull request #13216: Remove unused ZooKeeper log level configuration from `connect-log4j.properties`
Posted by "scholzj (via GitHub)" <gi...@apache.org>.
scholzj commented on PR #13216:
URL: https://github.com/apache/kafka/pull/13216#issuecomment-1439131216
@mimaison @fvaleri Done, thanks.
--
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