martinzink commented on a change in pull request #1225:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1225#discussion_r769575387
##########
File path: extensions/mqtt/processors/AbstractMQTTProcessor.h
##########
@@ -130,8 +128,8 @@ class AbstractMQTTProcessor : public core::Processor {
MQTTClient_deliveryToken delivered_token_;
std::string uri_;
std::string topic_;
- int64_t keepAliveInterval_;
- int64_t connectionTimeOut_;
+ std::chrono::seconds keepAliveInterval_{60};
+ std::chrono::seconds connectionTimeOut_{30};
Review comment:
wow good catch, I've changed so now we store ms, and explicitly cast to
seconds and ms where it is used in
https://github.com/apache/nifi-minifi-cpp/pull/1225/commits/78f8658fde802c73a5965dd734a544acbc37dca3
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
