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.

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

fixed MQ serverMode lost default topic #4981

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

enqbs
Copy link

@enqbs enqbs commented Dec 10, 2023

尊敬的作者您好,MQ模式启动触发以下场景问题。”默认topic和动态topic同时存在,指定需同步的数据库.数据表走指定的动态topic,未指定的数据库.数据表统一走默认topic“,结果经过源码DEBUG发现只要动态topic存在会把topic字符串中所有的”.“替换成”_“导致默认topic丢失。
配置:
无标题_03
修改前输出日志:
无标题
我的建议是动态topic与默认topic匹配直接返回默认topic。
修改后代码:
02
修改后输出日志:
无标题_01
希望得到采纳,感谢。

@CLAassistant
Copy link

CLAassistant commented Dec 10, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


52CL seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@agapple
Copy link
Member

agapple commented Dec 19, 2023

default也需要统一采用转义处理,避免不同MQ之间的差异

@enqbs
Copy link
Author

enqbs commented Dec 19, 2023

default也需要统一采用转义处理,避免不同MQ之间的差异

好的。虽然四个MQ我都改了,但保持原样我认为也没问题。

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.

None yet

3 participants