Skip to content

[RatisConsensus] retry cache expiration time should be longer than retriable-client wait duration#11045

Merged
OneSizeFitsQuorum merged 2 commits intoapache:masterfrom
SzyWilliam:retry_cache_0905
Sep 6, 2023
Merged

[RatisConsensus] retry cache expiration time should be longer than retriable-client wait duration#11045
OneSizeFitsQuorum merged 2 commits intoapache:masterfrom
SzyWilliam:retry_cache_0905

Conversation

@SzyWilliam
Copy link
Copy Markdown
Member

@SzyWilliam SzyWilliam changed the title [RatisConsensus] retry cache expiration time should be bigger than client wait duration [RatisConsensus] retry cache expiration time should be longer than retriable-client wait duration Sep 5, 2023
Copy link
Copy Markdown
Contributor

@OneSizeFitsQuorum OneSizeFitsQuorum left a comment

Choose a reason for hiding this comment

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

LGTM!

@SzyWilliam
Copy link
Copy Markdown
Member Author

The good news is that we can now rule out the apply twice problem and guarantee exactly-once semantic!

@OneSizeFitsQuorum OneSizeFitsQuorum merged commit b9ed1ed into apache:master Sep 6, 2023
SzyWilliam added a commit to SzyWilliam/iotdb that referenced this pull request Sep 6, 2023
OneSizeFitsQuorum pushed a commit that referenced this pull request Sep 6, 2023
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