Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

The wrong topic was consumed in the DefaultMessageStoreTest test program #2811

Closed
BurningCN opened this issue Apr 15, 2021 · 2 comments
Closed

The wrong topic was consumed in the DefaultMessageStoreTest test program #2811

BurningCN opened this issue Apr 15, 2021 · 2 comments
Labels
module/test type/bug
Milestone
4.9.0

Comments

@BurningCN
Copy link
Member

image
image

The putMessage uses the FooBar topic, but getMessage gets the message from the TOPIC_A topic
@duhenglucky duhenglucky added this to the 4.8.1 milestone Apr 19, 2021
@duhenglucky
Copy link
Contributor

@BurningCN Would you like submit a PR to fix it?

@BurningCN
Copy link
Member Author

@BurningCN Would you like submit a PR to fix it?

Okay, no problem

BurningCN added a commit to BurningCN/rocketmq that referenced this issue Apr 19, 2021
@BurningCN
[ISSUE apache#2811] Fix the wrong topic was consumed in the DefaultMe…
0b6bd18 
…ssageStoreTest test program
@BurningCN BurningCN mentioned this issue Apr 19, 2021
Merged
duhenglucky added a commit that referenced this issue Apr 19, 2021
@duhenglucky
Merge pull request #2824 from BurningCN/fix_test_consume_wrong_topic
8eb0472 
[ISSUE #2811] Fix the wrong topic was consumed in the DefaultMessageStoreTest test program
vongosling added a commit that referenced this issue May 19, 2021
@duhenglucky @ShadowySpirits
@panzhi33 @ArronHuang @vongosling @drgnchan @ph3636 @BurningCN @yuz10 @areyouok @ayanamist @zhangjidi2016
Fix conflict,merge develop (#2906)
ac9aaa4 
* [ISSUE #1233] Fix CVE-2011-1473

* fix Multiple instances in the same application share MQClientInstance

* [ISSUE #2748] Fix deleteSubscriptionGroup not remove consumer offset

* [ISSUE #2745] Changed the support time of the request/reply feature to 4.6.0.

Co-authored-by: von gosling <vongosling@apache.org>

* [ISSUE #2729] Replace with Math.min method call

* [ISSUE #2801]Fix NamesrvAddr connot set in Producer

* [ISSUE 2800] optimize: the spelling of topicSynFlag

Co-authored-by: ph3636 <tianxingguang@kanzhun.com>

* [ISSUE #2803] Fix the endpoint cannot get instanceId without http (#2804)

* fix the endpoint cannot get instanceId without http

* fix the endpoint cannot get instanceId without http

* add unit test

* add unit test

* add unit test

Co-authored-by: panzhi33 <wb-pz502261@alibaba-inc.com>

* fix messageArrivingListener NPE

* [ISSUE #2538]Optimize log output when message trace saving fails

* [ISSUE #2811] Fix the wrong topic was consumed in the DefaultMessageStoreTest test program

* [ISSUE #2821] Overriding the ServiceThread#shutdown in HAClient class

* [ISSUE #2805] remove redundant package imports

* [ISSUE #2833] Support trace for TranscationProducer (#2834)

* [ISSUE #2732] Fix message loss problem when rebalance with LitePullConsumer (#2832)

* [ISSUE #2732] Fix message loss problem when rebalance with LitePullConsumer

* Fix message loss problem when rebalance with LitePullConsumer, update 2

* [ISSUE #2846]fix -E might not port to other systems

* fix some nonconformity after checkstyle

* Support OpenTracing(#2861)

* [ISSUE #2872] remove log files created by integration test when mvn clean

* [ISSUE #2872] move log files created by integration test to target dir

* Change log level to debug: "Half offset {} has been committed/rolled back"

* Fix unit test stability

Bump mockito-core to 3.10.0, remove powermock dependency, suppress useless logging

* [ISSUE #2898] Resolve rocketmq-example project failed during checkstyle execution (#2899)

Co-authored-by: SSpirits <shadowyspirits@outlook.com>
Co-authored-by: panzhi33 <wb-pz502261@alibaba-inc.com>
Co-authored-by: panzhi <panzhi33@qq.com>
Co-authored-by: ArronHuang <41609451+ArronHuang@users.noreply.github.com>
Co-authored-by: von gosling <vongosling@apache.org>
Co-authored-by: drgnchan <40224023+drgnchan@users.noreply.github.com>
Co-authored-by: zhangjidi2016 <zhangjidi@cmss.chinamobile.com>
Co-authored-by: ph3636 <38041490+ph3636@users.noreply.github.com>
Co-authored-by: ph3636 <tianxingguang@kanzhun.com>
Co-authored-by: BurningCN <1015773611@qq.com>
Co-authored-by: francis lee <francislee.cn@outlook.com>
Co-authored-by: 灼华 <43363120+BurningCN@users.noreply.github.com>
Co-authored-by: yuz10 <845238369@qq.com>
Co-authored-by: huangli <areyouok@gmail.com>
Co-authored-by: chenrl <raymond2366@outlook.com>
Co-authored-by: ayanamist <ayanamist@gmail.com>
Co-authored-by: zhangjidi2016 <1017543663@qq.com>
@zongtanghu zongtanghu modified the milestones: 4.8.1, 4.9.0 Jun 4, 2021
GenerousMan pushed a commit to GenerousMan/rocketmq that referenced this issue Aug 12, 2022
@BurningCN
[ISSUE apache#2811] Fix the wrong topic was consumed in the DefaultMe…
e85619c 
…ssageStoreTest test program
GenerousMan pushed a commit to GenerousMan/rocketmq that referenced this issue Aug 12, 2022
@duhenglucky
Merge pull request apache#2824 from BurningCN/fix_test_consume_wrong_…
1f00037 
…topic

[ISSUE apache#2811] Fix the wrong topic was consumed in the DefaultMessageStoreTest test program
pulllock pushed a commit to pulllock/rocketmq that referenced this issue Oct 19, 2023
@BurningCN
[ISSUE apache#2811] Fix the wrong topic was consumed in the DefaultMe…
e8dc289 
…ssageStoreTest test program
pulllock pushed a commit to pulllock/rocketmq that referenced this issue Oct 19, 2023
@duhenglucky
Merge pull request apache#2824 from BurningCN/fix_test_consume_wrong_…
5aa8dee 
…topic

[ISSUE apache#2811] Fix the wrong topic was consumed in the DefaultMessageStoreTest test program
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
module/test type/bug
Projects
None yet
Milestone
4.9.0
Development

No branches or pull requests
3 participants
@zongtanghu @duhenglucky @BurningCN