Skip to content

[fix][broker][branch-3.0] Avoid consumers receiving acknowledged messages from compacted topic after reconnection (#21187)#22142

Merged
heesung-sohn merged 1 commit intoapache:branch-3.0from
coderzc:cherry-pick/3.0/21187
Mar 6, 2024
Merged

[fix][broker][branch-3.0] Avoid consumers receiving acknowledged messages from compacted topic after reconnection (#21187)#22142
heesung-sohn merged 1 commit intoapache:branch-3.0from
coderzc:cherry-pick/3.0/21187

Conversation

@coderzc
Copy link
Member

@coderzc coderzc commented Feb 28, 2024

Motivation & Modifications

Cherry-pick #21187 into branch-3.0

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository:

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Feb 28, 2024
@lhotari
Copy link
Member

lhotari commented Mar 5, 2024

closing and reopening to trigger CI

@lhotari lhotari closed this Mar 5, 2024
@lhotari lhotari reopened this Mar 5, 2024
@lhotari
Copy link
Member

lhotari commented Mar 5, 2024

@heesung-sn this is next on the list for branch-3.0

@heesung-sohn
Copy link
Contributor

heesung-sohn commented Mar 6, 2024

Found this error is from the current branch-3.0. merging this.

https://github.com/apache/pulsar/actions/runs/8178640132/job/22363372369

BrokerServiceAutoSubscriptionCreationTest.testDynamicConfigurationTopicAutoSubscriptionCreation: pulsar-broker/src/test/java/org/apache/pulsar/broker/service/BrokerServiceAutoSubscriptionCreationTest.java#L169
java.util.concurrent.ExecutionException: org.apache.pulsar.client.api.PulsarClientException$SubscriptionNotFoundException: {"errorMsg":"Subscription does not exist","reqId":1658404878796832891, "remote":"localhost/127.0.0.1:44393", "local":"/127.0.0.1:57930"}

@heesung-sohn heesung-sohn merged commit 5b62d4b into apache:branch-3.0 Mar 6, 2024
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 ready-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants