Skip to content

[Broker] Make PersistentTopicsBase#internalGetPartitionedMetadata async#14153

Merged
Technoboy- merged 12 commits into
apache:masterfrom
Technoboy-:getPartitionedMetadata-async
May 5, 2022
Merged

[Broker] Make PersistentTopicsBase#internalGetPartitionedMetadata async#14153
Technoboy- merged 12 commits into
apache:masterfrom
Technoboy-:getPartitionedMetadata-async

Conversation

@Technoboy-

Copy link
Copy Markdown
Contributor

Motivation

Make PersistentTopicsBase#internalGetPartitionedMetadata async.

Documentation

  • no-need-doc

@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Feb 7, 2022
RobertIndie
RobertIndie previously approved these changes Feb 8, 2022
@Technoboy-
Technoboy- force-pushed the getPartitionedMetadata-async branch from f1bba01 to 4c440c7 Compare February 8, 2022 05:34
Comment thread pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/AdminResource.java Outdated
Comment thread pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/AdminResource.java Outdated
@github-actions

Copy link
Copy Markdown

The pr had no activity for 30 days, mark with Stale label.

@Technoboy-
Technoboy- force-pushed the getPartitionedMetadata-async branch from 483fc01 to 9b96827 Compare May 1, 2022 04:17
@Technoboy- Technoboy- added type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages area/broker and removed lifecycle/stale labels May 1, 2022
@Technoboy- Technoboy- added this to the 2.11.0 milestone May 1, 2022

@mattisonchao mattisonchao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Overall LGTM, left some small comments.

Comment thread pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/AdminResource.java Outdated

@mattisonchao mattisonchao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Technoboy- Thanks for your answer

@eolivelli eolivelli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@nodece

nodece commented Oct 9, 2022

Copy link
Copy Markdown
Member

@Technoboy- Could you make a PR to branch-2.10?

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

Labels

area/broker cherry-picked/branch-2.10 doc-not-needed Your PR changes do not impact docs release/2.10.3 type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants