Skip to content

feat(version): add api to get SDK versions #261

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

Merged
merged 8 commits into from
Feb 27, 2020

Conversation

ShannonDing
Copy link
Member

@ShannonDing ShannonDing commented Feb 26, 2020

What is the purpose of the change

add api to get SDK versions
CPP API: version()
C API: ShowProducerVersion()... and so on.

you can get SDK version strings like this:
CPP CORE VERSION: 2.0.0, BUILD TIME: 19:30:30 02-14-2020

Brief changelog

feat(version): add api to get SDK versions

@ShannonDing ShannonDing added the enhancement New feature or request label Feb 26, 2020
@ShannonDing ShannonDing added this to the 2.0.1 milestone Feb 26, 2020
@codecov-io
Copy link

codecov-io commented Feb 26, 2020

Codecov Report

Merging #261 into master will increase coverage by 0.55%.
The diff coverage is 94.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #261      +/-   ##
==========================================
+ Coverage   57.86%   58.41%   +0.55%     
==========================================
  Files         182      182              
  Lines       11683    11769      +86     
==========================================
+ Hits         6760     6875     +115     
+ Misses       4923     4894      -29
Impacted Files Coverage Δ
src/protocol/ConsumerRunningInfo.h 100% <ø> (ø) ⬆️
src/consumer/DefaultMQPushConsumerImpl.h 0% <ø> (ø) ⬆️
test/src/extern/CPushConsumerTest.cpp 96.38% <100%> (+0.23%) ⬆️
test/src/MQClientAPIImpTest.cpp 94.73% <100%> (-0.72%) ⬇️
test/src/extern/CProducerTest.cpp 96.85% <100%> (+0.28%) ⬆️
test/src/extern/CPullConsumerTest.cpp 100% <100%> (ø) ⬆️
src/common/DefaultMQClient.cpp 71.02% <100%> (+0.83%) ⬆️
src/consumer/DefaultMQPullConsumer.cpp 40.78% <100%> (+6%) ⬆️
src/producer/DefaultMQProducer.cpp 41.28% <100%> (+4.02%) ⬆️
src/transport/TcpRemotingClient.cpp 9.26% <100%> (+0.51%) ⬆️
... and 16 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 9265ba4...8cc61f9. Read the comment docs.

#define ROCKETMQCPP_VERSION "2.0.0"
#define BUILD_DATE "02-14-2020"
#define BUILD_DATE "22:50:18 02-14-2020"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ToDo, hope to be fixed in the near long time.

void DefaultMQProducerImpl::logConfigs() {
showClientConfigs();

LOG_WARN("SendMsgTimeout:%d ms", m_sendMsgTimeout);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

warn is not better for normal output.

@vongosling vongosling merged commit 12a73b1 into apache:master Feb 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants