Github user mans2singh commented on the issue:
https://github.com/apache/nifi/pull/2012
Hello @jvwing - I've updated the code based on your review comments
(updated notice, refactored constants to base class).
Regarding the name Get vs Fetch - I am trying to use the terminology used
by RethinkDB and their apis. If you think that Fetch is more expressive and I
can change it. Also, I've added tag 'fetch' so that the processor will turn up
in searches.
I was interpolating id in case of empty id is because sometimes it might be
null. So, I've updated the code to check blank rather than just empty and have
also added unit tests for those scenarios.
Regarding 'not_found' relation - I wanted to give the users option to
handle this scenario separately than other errors. From what I understand, the
user may connect both 'failure' and 'not_found' to same component without
losing any flexibility.
Please let me know if you have any other recommendations/thoughts.
Thanks again.
Mans
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
