Skip to content
This repository was archived by the owner on Jan 24, 2024. It is now read-only.

AdminManager: handle properly async deletePartitionedTopic - #972

Merged
BewareMyPower merged 3 commits into
streamnative:masterfrom
eolivelli:fix/admin-async
Dec 22, 2021
Merged

AdminManager: handle properly async deletePartitionedTopic#972
BewareMyPower merged 3 commits into
streamnative:masterfrom
eolivelli:fix/admin-async

Conversation

@eolivelli

Copy link
Copy Markdown
Contributor

In AdminManager there are some points in which we are not using correctly CompletableFuture.
This patch fixes deletePartitionedTopic and a couple of other parts.

This patch is trivial rework, no need to add tests

@BewareMyPower BewareMyPower added type/enhancement Indicates an improvement to an existing feature release/2.9.1 and removed type/bug release/2.9.0 labels Dec 17, 2021
@eolivelli

Copy link
Copy Markdown
Contributor Author

patch updated @BewareMyPower

@BewareMyPower
BewareMyPower merged commit 540adeb into streamnative:master Dec 22, 2021
BewareMyPower pushed a commit that referenced this pull request Dec 29, 2021
In AdminManager there are some points in which we are not using correctly CompletableFuture.
This patch fixes deletePartitionedTopic and a couple of other parts.

This patch is trivial rework, no need to add tests
BewareMyPower pushed a commit that referenced this pull request Dec 29, 2021
In AdminManager there are some points in which we are not using correctly CompletableFuture.
This patch fixes deletePartitionedTopic and a couple of other parts.

This patch is trivial rework, no need to add tests
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants