Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KAFKA-16228: Add remote log metadata flag to the dump log tool #16475

Merged
merged 8 commits into from
Jul 10, 2024

Conversation

fvaleri
Copy link
Contributor

@fvaleri fvaleri commented Jun 27, 2024

This change adds the --remote-log-metadata-decoder flag to the kafka-dump-log.sh tool. This new flag can be used to decode the payload of the __remote_log_metadata records produced by the default RemoteLogMetadataManager.

This change adds the --remote-log-metadata-decoder flag to the kafka-dump-log.sh tool.
This new flag can be used to decode the payload of the __remote_log_metadata records produced by the default RemoteLogMetadataManager.

Signed-off-by: Federico Valeri <[email protected]>
@fvaleri fvaleri added tools tiered-storage Related to the Tiered Storage feature labels Jun 27, 2024
Signed-off-by: Federico Valeri <[email protected]>
Signed-off-by: Federico Valeri <[email protected]>
@fvaleri fvaleri requested a review from divijvaidya July 5, 2024 07:22
Copy link
Contributor

@showuon showuon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR. Left some comments.

@showuon
Copy link
Contributor

showuon commented Jul 5, 2024

And thanks for adding these tests!

@fvaleri fvaleri requested a review from showuon July 5, 2024 08:37
@fvaleri
Copy link
Contributor Author

fvaleri commented Jul 5, 2024

@divijvaidya @showuon thanks for the reviews. Your comments should be addressed now.

Copy link
Contributor

@showuon showuon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update. Some other comments.

Copy link
Contributor

@showuon showuon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for the improvement!

Signed-off-by: Federico Valeri <[email protected]>
@showuon
Copy link
Contributor

showuon commented Jul 10, 2024

Failed tests are unrelated.

@showuon showuon merged commit 8491e84 into apache:trunk Jul 10, 2024
1 check failed
@fvaleri fvaleri deleted the rlmm-dump branch July 10, 2024 05:34
abhi-ksolves pushed a commit to ksolves/kafka that referenced this pull request Jul 31, 2024
…e#16475)

This change adds the --remote-log-metadata-decoder flag to the kafka-dump-log.sh tool. This new flag can be used to decode the payload of the __remote_log_metadata records produced by the default RemoteLogMetadataManager.

Reviewers: Luke Chen <[email protected]>, Divij Vaidya <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tiered-storage Related to the Tiered Storage feature tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants