r-sidd commented on PR #7270: URL: https://github.com/apache/nifi/pull/7270#issuecomment-1561348649
> Thanks for the work on this @r-sidd. Although the unit tests pass, they use a mocked Ignite Processor that does not exercise actual behavior. > > The integration tests fail to run when attempting connect to the internal test server. > > The [Apache Ignite 2.0 Migration Guide](https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.0+Migration+Guide) highlights a number of important changes, so I'm not surprised this is failing. The current Processor implementations rely on a Spring Configuration file, which is not the best approach, and these Processors have not been maintained since the original contribution in 2016. With that background, I will plan on deprecating and removing them for NiFi 2.0 in a separate issue. Makes sense, thanks for the explanation 😄 -- 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]
