lordgamez commented on pull request #1001: URL: https://github.com/apache/nifi-minifi-cpp/pull/1001#issuecomment-779307903
I am okay with the change, but IMO we should also consider adding the compilation of the extension to one of our CI jobs as well to avoid regression in the future. That would unfortunately implicate further changes: - Bazel should be included in the bootstrap.sh in case Tensorflow is built - A CMake file for bundling Tensorflow should be added - Tensorflow then should be built locally which needs to be checked how long that may take in the CI environment ---------------------------------------------------------------- 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
