Skip to content

Conversation

@atheendre130505
Copy link

  • Add missing @return description in ProducerManager.publishMessage()
  • Add missing @return description in RemoteLogMetadataCache.listAllRemoteLogSegments()
  • Add missing @throws description in LogDirFailureChannel.takeNextOfflineLogDir()

All Javadoc tags now have proper descriptions following Apache Kafka documentation standards.

Delete this text and replace it with a detailed description of your change. The
PR title and body will become the squashed commit message.

If you would like to tag individuals, add some commentary, upload images, or
include other supplemental information that should not be part of the eventual
commit message, please use a separate comment.

If applicable, please include a summary of the testing strategy (including
rationale) for the proposed change. Unit and/or integration tests are expected
for any behavior change and system tests should be considered for larger
changes.

- Add missing @return description in ProducerManager.publishMessage()
- Add missing @return description in RemoteLogMetadataCache.listAllRemoteLogSegments()
- Add missing @throws description in LogDirFailureChannel.takeNextOfflineLogDir()

All Javadoc tags now have proper descriptions following Apache Kafka documentation standards.
@github-actions github-actions bot added triage PRs from the community storage Pull requests that target the storage module tiered-storage Related to the Tiered Storage feature small Small PRs labels Oct 26, 2025
@FrankYang0529
Copy link
Member

@atheendre130505 Thanks for the contribution. Could you follow the guide to modify PR title? Thanks.

https://cwiki.apache.org/confluence/display/KAFKA/Contributing+Code+Changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-approved small Small PRs storage Pull requests that target the storage module tiered-storage Related to the Tiered Storage feature triage PRs from the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants