Skip to content

[Broker] Make PersistentTopicsBase#internalExamineMessage async#14062

Closed
RobertIndie wants to merge 5 commits into
apache:masterfrom
RobertIndie:fix-examineMessage
Closed

[Broker] Make PersistentTopicsBase#internalExamineMessage async#14062
RobertIndie wants to merge 5 commits into
apache:masterfrom
RobertIndie:fix-examineMessage

Conversation

@RobertIndie

Copy link
Copy Markdown
Member

Master Issue: #14013

Motivation

See #14013

Modifications

  • Make PersistentTopicsBase#internalExamineMessage async

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

Documentation

  • no-need-doc

Motivation
See apache#14013

Modification
* Make PersistentTopicsBase#internalExamineMessage async

Signed-off-by: Zike Yang <zkyang@streamnative.io>
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Jan 30, 2022
Signed-off-by: Zike Yang <zkyang@streamnative.io>
Signed-off-by: Zike Yang <zkyang@streamnative.io>
Signed-off-by: Zike Yang <zkyang@streamnative.io>
@RobertIndie

Copy link
Copy Markdown
Member Author

/pulsarbot run-failure-checks

1 similar comment
@RobertIndie

Copy link
Copy Markdown
Member Author

/pulsarbot run-failure-checks

Signed-off-by: Zike Yang <zkyang@streamnative.io>
@github-actions

Copy link
Copy Markdown

The pr had no activity for 30 days, mark with Stale label.

@github-actions

Copy link
Copy Markdown

The pr had no activity for 30 days, mark with Stale label.

@RobertIndie

Copy link
Copy Markdown
Member Author

These changes of codes have been moved to #16226. Close this one.

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 lifecycle/stale Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants