feat(producer): add regionId support in the send result by ShannonDing · Pull Request #258 · apache/rocketmq-client-cpp

Conversation

@ShannonDing

Copy link

Member

What is the purpose of the change

feat(producer): add regionId support in the send result

@ShannonDing ShannonDing added the enhancement

New feature or request

label

Feb 24, 2020

@ShannonDing ShannonDing added this to the 2.0.1 milestone

Feb 24, 2020

@codecov-io

Copy link

Codecov Report

Merging #258 into master will increase coverage by 0.16%.
The diff coverage is 93.54%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #258      +/-   ##
==========================================
+ Coverage   57.69%   57.86%   +0.16%     
==========================================
  Files         182      182              
  Lines       11652    11680      +28     
==========================================
+ Hits         6723     6759      +36     
+ Misses       4929     4921       -8
Impacted Files Coverage Δ
include/SendResult.h 0% <ø> (ø) ⬆️
src/MQClientAPIImpl.cpp 24.48% <100%> (ø) ⬆️
src/protocol/CommandHeader.cpp 85.71% <100%> (+0.27%) ⬆️
test/src/protocol/CommandHeaderTest.cpp 100% <100%> (ø) ⬆️
test/src/producer/DefaultMQProducerImplTest.cpp 98.26% <100%> (+0.03%) ⬆️
src/protocol/CommandHeader.h 79.76% <100%> (+0.49%) ⬆️
src/producer/SendResult.cpp 95.65% <75%> (+21.96%) ⬆️

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 41d7c35...b7cd030. Read the comment docs.

@ShannonDing ShannonDing merged commit e0f05d6 into apache:master

Feb 24, 2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

enhancement

New feature or request

Projects

None yet

Milestone

2.0.1

Development

Successfully merging this pull request may close these issues.

2 participants

@ShannonDing @codecov-io