-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
支持amqp(RabbitMQ) #2156
支持amqp(RabbitMQ) #2156
Conversation
|
Codecov Report
@@ Coverage Diff @@
## master #2156 +/- ##
===========================================
- Coverage 6.15% 6.1% -0.06%
Complexity 197 197
===========================================
Files 277 281 +4
Lines 26988 27214 +226
Branches 3995 4013 +18
===========================================
Hits 1661 1661
- Misses 25063 25289 +226
Partials 264 264
Continue to review full report at Codecov.
|
tks |
public void init(MQProperties mqProperties) { | ||
this.mqProperties = mqProperties; | ||
ConnectionFactory factory = new ConnectionFactory(); | ||
factory.setHost(mqProperties.getServers()); |
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.
ConnectionFactory的host
和port
属性是分开设置的,这种构造方式存在问题
} | ||
factory.setVirtualHost(mqProperties.getVhost()); | ||
try { | ||
connect = factory.newConnection(); |
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.
connect = factory.newConnection(com.rabbitmq.client.Address.parseAddresses(mqProperties.getServers()));
tks |
It does not seem possible to modify the wiki through PR. The absence of documentation can be confusing to users. |
对rabbitmq的支持
topic设置规则等同于rocketMQ和kafka
server端 topic=routekey
adapter端 topic=queue
目前exchnage/routekey/topic的绑定关系需要自行在rabbitmq manager中完成
若填写阿里uid且填写了阿里的ak和sk。则默认为阿里云MQ模式。若不需要则需要留空哦