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
feat: refactor RocketMQ bridge to connector and action #12488
feat: refactor RocketMQ bridge to connector and action #12488
Conversation
a3c3310
to
e284a83
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: might be good to have some smoke tests that go through the v2 APIs and schemas (like the ones in emqx_bridge_v2_testlib
).
apps/emqx_bridge_rocketmq/src/emqx_bridge_rocketmq_connector.erl
Outdated
Show resolved
Hide resolved
Thanks @thalesmg for a good review. I fixed the issues that you pointed out. |
Thanks. I will have a look at emqx_bridge_v2_testlib. If we add such tests I think we can do it in another PR so we can get this merged. |
on_get_channel_status( | ||
_ResId, | ||
_ChannelId, | ||
_State | ||
) -> | ||
?status_connected. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
check channel ID exist.
Fixes:
https://emqx.atlassian.net/browse/EMQX-11467
Release version: v/e5.?
Summary
PR Checklist
Please convert it to a draft if any of the following conditions are not met. Reviewers may skip over until all the items are checked:
changes/(ce|ee)/(feat|perf|fix|breaking)-<PR-id>.en.md
files