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

Fixed missed ZK caching when fetching list of namespaces for a tenant #10594

Merged
merged 2 commits into from
May 17, 2021

Conversation

merlimat
Copy link
Contributor

Motivation

The ZK reads triggered by the "get namespaces of a tenant" admin operation are being issued directly on the ZK client and therefore are not getting cached by brokers.

This change is only for 2.7 branch. The issue is not present in current master since the code was already ported to use MetadataStore API.

@merlimat merlimat added type/bug The PR fixed a bug or issue reported a bug release/2.7.3 labels May 15, 2021
@merlimat merlimat self-assigned this May 15, 2021
Copy link
Contributor

@addisonj addisonj left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

Lgtm

@sijie sijie added this to the 2.8.0 milestone May 15, 2021
@merlimat merlimat merged commit 883e0d4 into apache:branch-2.7 May 17, 2021
@merlimat merlimat deleted the fix-get-namespaces-list branch May 17, 2021 21:09
eolivelli pushed a commit to datastax/pulsar that referenced this pull request May 18, 2021
…apache#10594)

* Fixed missed ZK caching when fetching list of namespaces for a tenant

* Fixed generic object type

(cherry picked from commit 883e0d4)
@codelipenghui codelipenghui added the cherry-picked/branch-2.7 Archived: 2.7 is end of life label May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-picked/branch-2.7 Archived: 2.7 is end of life release/2.7.3 type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants