Github user phrocker commented on the issue:
https://github.com/apache/nifi-minifi-cpp/pull/37
@jdye64 feel free to close this. Thanks!
---
Github user achristianson commented on the issue:
https://github.com/apache/nifi-minifi-cpp/pull/37
Taking this on.
---
Github user apiri commented on the issue:
https://github.com/apache/nifi-minifi-cpp/pull/37
@jdye64 Certainly in favor for the inclusion and support for dynamic
properties. Some of that work in Processor could be used toward that
functionality. Would obviously need to adapt things a
Github user jdye64 commented on the issue:
https://github.com/apache/nifi-minifi-cpp/pull/37
@apiri so do you think we should just close this PR then? Trying to tidy up
a few PRs this morning
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user apiri commented on the issue:
https://github.com/apache/nifi-minifi-cpp/pull/37
@jdye64 Didn't mean to leave things hanging. My thought process is still
on removing the configuration option and letting each processor choose how/if
to interpret dynamic properties. With acc
Github user jdye64 commented on the issue:
https://github.com/apache/nifi-minifi-cpp/pull/37
@apiri Where do you think we stand on this? I know you had some concerns
about the implementation approach in the beginning. Is that still the case? If
so what can we do to resolve those?
--
Github user jdye64 commented on the issue:
https://github.com/apache/nifi-minifi-cpp/pull/37
@phrocker thanks a ton for the suggestions. Updated with your
recommendations.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user apiri commented on the issue:
https://github.com/apache/nifi-minifi-cpp/pull/37
not discounting the possibility, but with access to the YAML, there are
other potential angles that are also at hand of a similar nature. I do think
the impact can certainly be mitigated durin
Github user jdye64 commented on the issue:
https://github.com/apache/nifi-minifi-cpp/pull/37
@apiri I agree. That was my intent was to allow each Processor itself to
determine how/if it used the dynamic properties that was the reason for
creating the new set in Processor.cpp for the d
