Skip to content

RATIS-1802. GrpcServerProtocolService encounters IllegalStateException: call already closed.#839

Merged
SzyWilliam merged 1 commit intoapache:masterfrom
szetszwo:RATIS-1802
Mar 6, 2023
Merged

RATIS-1802. GrpcServerProtocolService encounters IllegalStateException: call already closed.#839
SzyWilliam merged 1 commit intoapache:masterfrom
szetszwo:RATIS-1802

Conversation

@szetszwo
Copy link
Contributor

@szetszwo szetszwo commented Mar 3, 2023

@szetszwo
Copy link
Contributor Author

szetszwo commented Mar 3, 2023

  • Check close before onError.
  • Also remove a unnecessary stream in readIndex proto definition.

Copy link
Member

@SzyWilliam SzyWilliam left a comment

Choose a reason for hiding this comment

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

+1 the change looks good to me. Thanks a lot @szetszwo

@SzyWilliam SzyWilliam merged commit 8a4d60c into apache:master Mar 6, 2023
kaijchen added a commit to kaijchen/ratis that referenced this pull request Mar 7, 2023
…Exception: call already closed. (apache#839)"

This reverts commit 8a4d60c.
adoroszlai pushed a commit that referenced this pull request Mar 9, 2023
…n: call already closed. (#839)

(cherry picked from commit 8a4d60c)
symious pushed a commit to symious/ratis that referenced this pull request Mar 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants