You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "showuon (via GitHub)" <gi...@apache.org> on 2023/05/25 09:51:45 UTC

[GitHub] [kafka] showuon opened a new pull request, #13763: MINOR: use debug level to log handleCommit

showuon opened a new pull request, #13763:
URL: https://github.com/apache/kafka/pull/13763

   In this [PR](https://github.com/apache/kafka/pull/13462/files#diff-38a0aec01732a73b638b91b3d88dfb6a074a6e47fac7972d5a8e7dc97c258606R321), we improved the log to make it clear, and backported to 3.4 branch. But in the [backport commit](https://github.com/apache/kafka/pull/13462/files#diff-38a0aec01732a73b638b91b3d88dfb6a074a6e47fac7972d5a8e7dc97c258606R321), we accidentally change the debug level to info level, and prefixed as `handleSnapshot`, instead of `handleCommit`. Fix it to avoid log flooding.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
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] showuon commented on pull request #13763: MINOR: use debug level to log handleCommit

Posted by "showuon (via GitHub)" <gi...@apache.org>.
showuon commented on PR #13763:
URL: https://github.com/apache/kafka/pull/13763#issuecomment-1563700586

   Failed tests are unrelated
   ```
       Build / JDK 11 and Scala 2.13 / kafka.server.DynamicBrokerReconfigurationTest.testTrustStoreAlter(String).quorum=kraft
       Build / JDK 8 and Scala 2.12 / kafka.admin.TopicCommandIntegrationTest.testDescribeAndListTopicsWithoutInternalTopics(String).quorum=zk
       Build / JDK 17 and Scala 2.13 / kafka.admin.ResetConsumerGroupOffsetTest.testResetOffsetsToZonedDateTime()
       Build / JDK 17 and Scala 2.13 / org.apache.kafka.tools.MetadataQuorumCommandTest.[1] Type=Raft-CoReside, Name=testDescribeQuorumStatusSuccessful, MetadataVersion=3.4-IV0, Security=PLAINTEXT
       Build / JDK 17 and Scala 2.13 / org.apache.kafka.trogdor.coordinator.CoordinatorTest.testTaskRequestWithOldStartMsGetsUpdated()
   ```


-- 
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] showuon merged pull request #13763: MINOR: use debug level to log handleCommit

Posted by "showuon (via GitHub)" <gi...@apache.org>.
showuon merged PR #13763:
URL: https://github.com/apache/kafka/pull/13763


-- 
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