[GitHub] [kafka] cyrusv commented on pull request #8918: Use debug level logging for noisy log messages in Connect

2020-10-15 Thread GitBox
cyrusv commented on pull request #8918: URL: https://github.com/apache/kafka/pull/8918#issuecomment-709483443 @rhauch, thanks for the review, I've made the requested changes. This is an automated message from the Apache Git

[GitHub] [kafka] cyrusv commented on pull request #8918: Use debug level logging for noisy log messages in Connect

2020-07-07 Thread GitBox
cyrusv commented on pull request #8918: URL: https://github.com/apache/kafka/pull/8918#issuecomment-655285115 @kkonstantine or @rhauch -- This has some traction and nonbinding approval -- I think we're ready for committer review?

[GitHub] [kafka] cyrusv commented on pull request #8918: Use debug level logging for noisy log messages in Connect

2020-07-07 Thread GitBox
cyrusv commented on pull request #8918: URL: https://github.com/apache/kafka/pull/8918#issuecomment-655277982 @C0urante, @wicknicks -- I downgraded two of the SourceTask even further to Trace, so only the central log statement is debug level. @wicknicks I see your point about dedup

[GitHub] [kafka] cyrusv commented on pull request #8918: Use debug level logging for noisy log messages in Connect

2020-07-06 Thread GitBox
cyrusv commented on pull request #8918: URL: https://github.com/apache/kafka/pull/8918#issuecomment-654549765 @wicknicks, @C0urante, @ryannedolan: Any feedback on this one? It will make logs on many systems much less expensive in the normal case.