lordgamez commented on a change in pull request #959:
URL: https://github.com/apache/nifi-minifi-cpp/pull/959#discussion_r543454567



##########
File path: extensions/standard-processors/processors/GetTCP.cpp
##########
@@ -148,7 +149,13 @@ void GetTCP::onSchedule(const 
std::shared_ptr<core::ProcessContext> &context, co
 
   logger_->log_trace("EOM is defined as %i", endOfMessageByte);
 
-  context->getProperty(ReconnectInterval.getName(), reconnect_interval_);
+  std::string reconnect_interval_str;
+  if (context->getProperty(ReconnectInterval.getName(), 
reconnect_interval_str) &&
+      core::Property::getTimeMSFromString(reconnect_interval_str, 
reconnect_interval_)) {
+    logger_->log_debug("Reconnect interval is %d ms", reconnect_interval_);

Review comment:
       Done in 
[c0110dc](https://github.com/apache/nifi-minifi-cpp/pull/959/commits/c0110dca0937ac4af93e32d7ccab8347132183c7)

##########
File path: extensions/rocksdb-repos/FlowFileRepository.h
##########
@@ -61,10 +61,11 @@ class FlowFileRepository : public core::Repository, public 
std::enable_shared_fr
       : FlowFileRepository(name) {
   }
 
-  FlowFileRepository(const std::string repo_name = "", std::string directory = 
FLOWFILE_REPOSITORY_DIRECTORY, int64_t maxPartitionMillis = 
MAX_FLOWFILE_REPOSITORY_ENTRY_LIFE_TIME,
+  FlowFileRepository(const std::string repo_name = "", std::string 
checkpoint_dir = FLOWFILE_CHECKPOINT_DIRECTORY, std::string directory = 
FLOWFILE_REPOSITORY_DIRECTORY, int64_t maxPartitionMillis = 
MAX_FLOWFILE_REPOSITORY_ENTRY_LIFE_TIME,

Review comment:
       Done in 
[c0110dc](https://github.com/apache/nifi-minifi-cpp/pull/959/commits/c0110dca0937ac4af93e32d7ccab8347132183c7)




----------------------------------------------------------------
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


Reply via email to