Skip to content

[fix] [broker] When notify wait opReadEntry, use same policy to calculate readPosition.#15102

Closed
horizonzy wants to merge 1 commit into
apache:masterfrom
horizonzy:fix-op-read-entry-pacth-2
Closed

[fix] [broker] When notify wait opReadEntry, use same policy to calculate readPosition.#15102
horizonzy wants to merge 1 commit into
apache:masterfrom
horizonzy:fix-op-read-entry-pacth-2

Conversation

@horizonzy

@horizonzy horizonzy commented Apr 10, 2022

Copy link
Copy Markdown
Member

Fixes #15101

Motivation

Unify opReadEntry's readPosition calculated policy.

Documentation

Need to update docs?

  • no-need-doc
    (Please explain why)

@github-actions

Copy link
Copy Markdown

@horizonzy:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@github-actions

Copy link
Copy Markdown

@horizonzy:Thanks for providing doc info!

@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Apr 10, 2022
@Jason918
Jason918 self-requested a review April 11, 2022 03:11
@horizonzy

Copy link
Copy Markdown
Member Author

close it, it cover by 23b3dd7 in #15098

@horizonzy horizonzy closed this Apr 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When notify wait opReadEntry, the readPosition calculated policy is different from opReadEntry.create()

1 participant