thenatog commented on PR #6154: URL: https://github.com/apache/nifi/pull/6154#issuecomment-1197222761
After some discussion offline I think we're looking pretty good here. We don't need to worry about merging the state in the ReportingTaskEntityMerger because there is no intermediate state (ie. it only uses RUNNING, STOPPED, DISABLED for state and status, and we don't need to resolve a state/status for this). The other code cleanliness issues have been addressed. Currently waiting for Github actions to complete. -- 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]
