Skip to content

[fix][flaky-test] testAutoSubscribePatternConsumer #16599

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.

Sign up for GitHub

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

Jump to bottom
Merged
Technoboy- merged 2 commits into from
Jul 15, 2022
Merged

[fix][flaky-test] testAutoSubscribePatternConsumer #16599

Technoboy- merged 2 commits into from
Jul 15, 2022

Conversation

@gaozhangmin
Copy link
Contributor

@gaozhangmin gaozhangmin commented Jul 14, 2022 •
edited
Loading

Motivation

Fixes #16595

Modifications

doReturn(CompletableFuture.completedFuture(topicNames)).when(nss)
                .getListOfPersistentTopics(NamespaceName.get("my-property/my-ns"));

Documentation

  • doc-not-needed

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Jul 14, 2022
@nodece
Copy link
Member

nodece commented Jul 14, 2022

The CI still failed.

gavingaozhangmin added 2 commits July 14, 2022 20:51
@Technoboy- Technoboy- added this to the 2.11.0 milestone Jul 15, 2022
@Technoboy- Technoboy- merged commit cfabdf8 into apache:master Jul 15, 2022
This was referenced Jul 15, 2022
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@HQebupt HQebupt HQebupt approved these changes

@Technoboy- Technoboy- Technoboy- approved these changes

@AnonHxy AnonHxy AnonHxy approved these changes

@nodece nodece nodece approved these changes

@Demogorgon314 Demogorgon314 Demogorgon314 approved these changes

Assignees

@gaozhangmin gaozhangmin

Labels

doc-not-needed Your PR changes do not impact docs type/flaky-tests

Projects

None yet

Milestone

2.11.0

Development

Successfully merging this pull request may close these issues.

Flaky-test: org.apache.pulsar.client.impl.PatternTopicsConsumerImplTest.testAutoSubscribePatternConsumer

6 participants

@gaozhangmin @nodece @HQebupt @Technoboy- @AnonHxy @Demogorgon314