Skip to content

Conversation

@congbobo184
Copy link
Contributor

@congbobo184 congbobo184 commented Nov 26, 2022

  1. cherry-pick: [bugfix] ManagedLedger: move to FENCED state in case of BadVersionException #17736 problem fix

  2. fix cherry-pick 17526 bugs
    https://github.com/poorbarcode/pulsar/blob/4db04f509451dc1f17b96009b2f5e268c1ea644b/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java#L2015
    it should change to !createTopicFuture.isPresent() not createTopicFuture.isPresent()

  3. and fix import

  4. fix cherry-pick https://github.com/apache/pulsar/pull18283 testDeleteTopicAndSchemaForV1, throw exception messages error

  5. run branch-2.9 test

Motivation

Documentation

  • doc-not-needed

Matching PR in forked repository

@congbobo184 congbobo184 changed the title [cherry-pick][branch-2.9] cherry-pick #17736 problem [cherry-pick][branch-2.9] cherry-pick #17736 problem and license header Nov 26, 2022
@congbobo184 congbobo184 changed the title [cherry-pick][branch-2.9] cherry-pick #17736 problem and license header [cherry-pick][branch-2.9] cherry-pick #17736 problem and DnsResolverTest license header Nov 26, 2022
@codelipenghui
Copy link
Contributor

codelipenghui commented Nov 27, 2022

The test testTruncateCorruptDataLedger is introduced by #16857. It should not belongs to branch-2.9.

@codelipenghui
Copy link
Contributor

/pulsarbot run-failure-checks

congbobo184 added 3 commits November 27, 2022 15:04
congbobo184 added 3 commits November 27, 2022 15:52
…ick-2.9/fix_#17736_problem_MessageCumulativeAckTest_mock
…ick-2.9/fix_#17736_problem_MessageCumulativeAckTest_mock
@congbobo184 congbobo184 merged commit 1c6abae into branch-2.9 Nov 27, 2022
@congbobo184 congbobo184 deleted the congbo/cherry-pick-2.9/fix_#17736_problem_MessageCumulativeAckTest_mock branch November 27, 2022 15:59
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.

5 participants