lordgamez commented on a change in pull request #1082: URL: https://github.com/apache/nifi-minifi-cpp/pull/1082#discussion_r640704122
########## File path: extensions/librdkafka/tests/ConsumeKafkaTests.cpp ########## @@ -422,7 +422,7 @@ class ConsumeKafkaContinuousPublishingTest : public ConsumeKafkaTest { int64_t value; org::apache::nifi::minifi::core::TimeUnit unit; REQUIRE(org::apache::nifi::minifi::core::Property::StringToTime(property_string, value, unit)); - int64_t value_as_ms; + int64_t value_as_ms = 0; Review comment: You are right, it just came up as an error while compiling and I fixed it on the fly. I removed it and will have it in a separate PR. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org