ali-sdk / ali-ons

Aliyun Open Notification Service Client
MIT License
151 stars 43 forks source link

feat: support multiple mq configs in the same client #85

Closed harry75369 closed 5 years ago

codecov[bot] commented 5 years ago

Codecov Report

Merging #85 into master will decrease coverage by 32.9%. The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #85       +/-   ##
===========================================
- Coverage   92.34%   59.43%   -32.91%     
===========================================
  Files          35       35               
  Lines        1816     1817        +1     
===========================================
- Hits         1677     1080      -597     
- Misses        139      737      +598
Impacted Files Coverage Δ
lib/mq_client.js 53.6% <100%> (-35.36%) :arrow_down:
lib/store/remote_broker.js 15.71% <0%> (-81.43%) :arrow_down:
lib/producer/topic_publish_info.js 35.29% <0%> (-58.83%) :arrow_down:
lib/process_queue.js 20.83% <0%> (-58.34%) :arrow_down:
lib/producer/mq_producer.js 41.4% <0%> (-53.91%) :arrow_down:
lib/mq_client_api.js 24.18% <0%> (-53.03%) :arrow_down:
lib/consumer/mq_push_consumer.js 41.92% <0%> (-52.41%) :arrow_down:
lib/channel.js 50% <0%> (-50%) :arrow_down:
lib/message/message.js 91.11% <0%> (-8.89%) :arrow_down:
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b62f120...fa9c6c5. Read the comment docs.

codecov[bot] commented 5 years ago

Codecov Report

Merging #85 into master will decrease coverage by 32.9%. The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #85       +/-   ##
===========================================
- Coverage   92.34%   59.43%   -32.91%     
===========================================
  Files          35       35               
  Lines        1816     1817        +1     
===========================================
- Hits         1677     1080      -597     
- Misses        139      737      +598
Impacted Files Coverage Δ
lib/mq_client.js 53.6% <100%> (-35.36%) :arrow_down:
lib/store/remote_broker.js 15.71% <0%> (-81.43%) :arrow_down:
lib/producer/topic_publish_info.js 35.29% <0%> (-58.83%) :arrow_down:
lib/process_queue.js 20.83% <0%> (-58.34%) :arrow_down:
lib/producer/mq_producer.js 41.4% <0%> (-53.91%) :arrow_down:
lib/mq_client_api.js 24.18% <0%> (-53.03%) :arrow_down:
lib/consumer/mq_push_consumer.js 41.92% <0%> (-52.41%) :arrow_down:
lib/channel.js 50% <0%> (-50%) :arrow_down:
lib/message/message.js 91.11% <0%> (-8.89%) :arrow_down:
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b62f120...fa9c6c5. Read the comment docs.

denghongcai commented 5 years ago

https://github.com/ali-sdk/ali-ons/blob/master/lib/client_config.js#L40 配置里改 instanceName 也可以吧

denghongcai commented 5 years ago

沟通后关闭