Skip to content

Conversation

@TaiJuWu
Copy link
Owner

@TaiJuWu TaiJuWu commented Nov 18, 2025

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.

// Merge topicIds from the new cluster with existing topicIds
// This is necessary because global metadata requests don't include topic info,
// but lookup requests do. We want to preserve topicId information from lookups.
Map<String, Uuid> mergedTopicIds = new HashMap<>(this.cluster.topicIds());
Copy link
Owner Author

Choose a reason for hiding this comment

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

Is overhead large?

@TaiJuWu TaiJuWu force-pushed the trunk branch 2 times, most recently from 03dce5e to 18dbd52 Compare December 3, 2025 03:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants