szaszm commented on code in PR #1918:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1918#discussion_r1949515438
##########
minifi-api/include/minifi-cpp/core/ProcessSession.h:
##########
@@ -97,6 +97,8 @@ class ProcessSession : public virtual ReferenceContainer {
// import from the data source.
virtual void import(const std::string& source, const
std::shared_ptr<core::FlowFile> &flow, bool keepSource = true, uint64_t offset
= 0) = 0;
+ virtual void import(const std::string& source,
std::vector<std::shared_ptr<FlowFile>> &flows, uint64_t offset, char
inputDelimiter) = 0;
+ virtual void import(const std::string& source,
std::vector<std::shared_ptr<FlowFile>> &flows, bool keepSource, uint64_t
offset, char inputDelimiter) = 0;
Review Comment:
Why are these added?
##########
minifi-api/include/minifi-cpp/core/ProcessorMetrics.h:
##########
@@ -34,23 +34,24 @@ class ProcessorMetrics : public virtual
state::response::ResponseNode {
virtual std::chrono::milliseconds getAverageSessionCommitRuntime() const = 0;
virtual std::chrono::milliseconds getLastSessionCommitRuntime() const = 0;
virtual void addLastSessionCommitRuntime(std::chrono::milliseconds runtime)
= 0;
+ virtual std::optional<size_t>
getTransferredFlowFilesToRelationshipCount(const std::string& relationship)
const = 0;
virtual std::atomic<size_t>& invocations() = 0;
virtual const std::atomic<size_t>& invocations() const = 0;
- virtual std::atomic<size_t>& incoming_flow_files() = 0;
- virtual const std::atomic<size_t>& incoming_flow_files() const = 0;
- virtual std::atomic<size_t>& transferred_flow_files() = 0;
- virtual const std::atomic<size_t>& transferred_flow_files() const = 0;
- virtual std::atomic<uint64_t>& incoming_bytes() = 0;
- virtual const std::atomic<uint64_t>& incoming_bytes() const = 0;
- virtual std::atomic<uint64_t>& transferred_bytes() = 0;
- virtual const std::atomic<uint64_t>& transferred_bytes() const = 0;
- virtual std::atomic<uint64_t>& bytes_read() = 0;
- virtual const std::atomic<uint64_t>& bytes_read() const = 0;
- virtual std::atomic<uint64_t>& bytes_written() = 0;
- virtual const std::atomic<uint64_t>& bytes_written() const = 0;
- virtual std::atomic<uint64_t>& processing_nanos() = 0;
- virtual const std::atomic<uint64_t>& processing_nanos() const = 0;
+ virtual std::atomic<size_t>& incomingFlowFiles() = 0;
+ virtual const std::atomic<size_t>& incomingFlowFiles() const = 0;
+ virtual std::atomic<size_t>& transferredFlowFiles() = 0;
+ virtual const std::atomic<size_t>& transferredFlowFiles() const = 0;
+ virtual std::atomic<uint64_t>& incomingBytes() = 0;
+ virtual const std::atomic<uint64_t>& incomingBytes() const = 0;
+ virtual std::atomic<uint64_t>& transferredBytes() = 0;
+ virtual const std::atomic<uint64_t>& transferredBytes() const = 0;
+ virtual std::atomic<uint64_t>& bytesRead() = 0;
+ virtual const std::atomic<uint64_t>& bytesRead() const = 0;
+ virtual std::atomic<uint64_t>& bytesWritten() = 0;
+ virtual const std::atomic<uint64_t>& bytesWritten() const = 0;
+ virtual std::atomic<uint64_t>& processingNanos() = 0;
+ virtual const std::atomic<uint64_t>& processingNanos() const = 0;
Review Comment:
These are already changed on the minifi-api branch, consider rebasing
##########
minifi-api/include/minifi-cpp/core/Processor.h:
##########
@@ -83,6 +83,8 @@ class Processor : public virtual Connectable, public virtual
ConfigurableCompone
virtual void validateAnnotations() const = 0;
virtual annotation::Input getInputRequirement() const = 0;
virtual gsl::not_null<std::shared_ptr<ProcessorMetrics>> getMetrics() const
= 0;
+ virtual std::string getProcessGroupUUIDStr() const = 0;
+ virtual void setProcessGroupUUIDStr(const std::string &uuid) = 0;
Review Comment:
Why are these added?
--
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]
