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

return if serviceNotReady while creating topic #336

Merged
merged 1 commit into from
Apr 6, 2017

Conversation

rdhabalia
Copy link
Contributor

Motivation

while creating topic : completes future with exception and return if bundle is not ready.

Modifications

return with failed result future if bundle is disable.

Result

It will avoid topic creation if bundle is disable.

@rdhabalia rdhabalia added the type/bug The PR fixed a bug or issue reported a bug label Apr 6, 2017
@rdhabalia rdhabalia added this to the 1.18 milestone Apr 6, 2017
@rdhabalia rdhabalia self-assigned this Apr 6, 2017
@merlimat merlimat merged commit 4866563 into apache:master Apr 6, 2017
@rdhabalia rdhabalia deleted the busyService branch June 21, 2017 18:54
hangc0276 pushed a commit to hangc0276/pulsar that referenced this pull request May 26, 2021
Fixes apache#336

Besides applying the LightProto generated new package name and some new APIs, this PR also fixes the tests error caused by apache#9240, which introduces a new method PulsarService#createLocalMetadataStore.

And some tests are affected by the wrong parse of entry data after the pulsar update, this PR fixes the test failures.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

3 participants