Skip to content

STORM-3879: Add Kafka README files to distribution#8435

Merged
reiabreu merged 1 commit intoapache:masterfrom
MNkulkarni06:STORM-3879-clean-final
Mar 22, 2026
Merged

STORM-3879: Add Kafka README files to distribution#8435
reiabreu merged 1 commit intoapache:masterfrom
MNkulkarni06:STORM-3879-clean-final

Conversation

@MNkulkarni06
Copy link
Contributor

What is the purpose of the change

This change ensures that the Kafka-related README files are included in the Storm distribution package.

Previously, the following README files were not part of the distribution:

  • external/storm-kafka-migration/README.md
  • external/storm-kafka-monitor/README.md

Including these files improves usability by providing necessary Kafka-related documentation directly within the distributed package.

How was the change tested

  • Verified that only storm-dist/source/src/main/assembly/source.xml is modified.
  • Checked using git diff upstream/master to ensure no unrelated changes are included.
  • Validated that the fileSet entries correctly reference the Kafka README paths.
  • Ensured that the configuration will include these files during the distribution build.

@MNkulkarni06
Copy link
Contributor Author

Hi, this PR addresses STORM-3879 by adding Kafka README files to the distribution. Please let me know if any changes are needed.

@MNkulkarni06
Copy link
Contributor Author

@rzo1 Could you please review this PR?

@MNkulkarni06
Copy link
Contributor Author

MNkulkarni06 commented Mar 22, 2026

@reiabreu Could you please review and if it is correct, then merge this PR?

@reiabreu reiabreu merged commit 40cf06a into apache:master Mar 22, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants