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.

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

Java client fails to describe partitions that are DEAD #9387

Closed
lenaschoenburg opened this issue May 16, 2022 · 0 comments · Fixed by #9410
Closed

Java client fails to describe partitions that are DEAD #9387

lenaschoenburg opened this issue May 16, 2022 · 0 comments · Fixed by #9410
Labels
kind/bug Categorizes an issue or PR as a bug scope/clients-java Marks an issue or PR to appear in the Java client section of the changelog version:1.3.9 version:8.1.0-alpha2 version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0

Comments

@lenaschoenburg
Copy link
Member

Describe the bug

PartitionInfoImpl only accepts partitions that are HEALTHY or UNHEALTHY and throws an exception for partitions which are DEAD.

https://github.com/camunda/zeebe/blob/1edfe4f762ef700885859bf87e1370963d3448a1/clients/java/src/main/java/io/camunda/zeebe/client/impl/response/PartitionInfoImpl.java#L46-L56

Expected behavior

PartitionInfoImpl should also support the DEAD status and not throw an exception.
@lenaschoenburg lenaschoenburg added kind/bug Categorizes an issue or PR as a bug scope/clients-java Marks an issue or PR to appear in the Java client section of the changelog team/process-automation labels May 16, 2022
@lenaschoenburg lenaschoenburg mentioned this issue May 18, 2022
Merged
@ghost ghost closed this as completed in e0ba2b3 May 23, 2022
This was referenced May 23, 2022
Merged
Merged
ghost pushed a commit that referenced this issue May 24, 2022
@lenaschoenburg
merge: #9435
54d3ada 
9435: [Backport stable/1.3] Java client accepts topologies with `DEAD` partitions r=saig0 a=backport-action

# Description
Backport of #9410 to `stable/1.3`.

relates to #9387

Co-authored-by: Ole Schönburg <ole.schoenburg@gmail.com>
ghost pushed a commit that referenced this issue May 24, 2022
@lenaschoenburg
merge: #9436
9c28a36 
9436: [Backport stable/8.0] Java client accepts topologies with `DEAD` partitions r=saig0 a=backport-action

# Description
Backport of #9410 to `stable/8.0`.

relates to #9387

Co-authored-by: Ole Schönburg <ole.schoenburg@gmail.com>
@ChrisKujawa ChrisKujawa added the version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0 label Oct 4, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug Categorizes an issue or PR as a bug scope/clients-java Marks an issue or PR to appear in the Java client section of the changelog version:1.3.9 version:8.1.0-alpha2 version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Java client accepts topologies with DEAD partitions camunda/camunda
4 participants
@npepinpe @lenaschoenburg @ChrisKujawa @remcowesterhoud