adamdebreceni commented on a change in pull request #1042:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1042#discussion_r609396538
##########
File path: libminifi/src/ThreadedSchedulingAgent.cpp
##########
@@ -88,6 +88,7 @@ void
ThreadedSchedulingAgent::schedule(std::shared_ptr<core::Processor> processo
auto sessionFactory =
std::make_shared<core::ProcessSessionFactory>(processContext);
+ processor->validateAnnotations();
Review comment:
given our direction towards all kinds of flow-verification, this could
be moved into either a `FlowController::verifyFlow(ProcessGroup&)` or
`ProcessGroup::verify` (the former might be better as then we could access the
services and repositories all at once) which could then be expanded as needed
(calling it from `FlowController::load`)
--
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:
[email protected]
