pulsar icon indicating copy to clipboard operation
pulsar copied to clipboard

Revert "Handle unknown runtime exception while reading entries (#2993)"

Open mattisonchao opened this issue 3 years ago • 1 comments

This revert commit 2c8c288a

Fixes https://github.com/apache/pulsar/issues/16907

Motivation

I think the problem described in #2993 is already solved by https://github.com/apache/pulsar/pull/2995, we don't need this workaround fix anymore.

Also, this workaround fix will introduce another problem and I will add a comment to the code.

Modifications

  • Revert PR #2993

Verifying this change

  • [x] Make sure that the change passes the CI checks.

  • [x] doc-not-needed

mattisonchao avatar Nov 01 '22 09:11 mattisonchao

@mattisonchao Please add the following content to your PR description and select a checkbox:

- [ ] `doc` <!-- Your PR contains doc changes -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->

github-actions[bot] avatar Nov 01 '22 09:11 github-actions[bot]