Github user mattyb149 commented on the issue:
https://github.com/apache/nifi/pull/2123
I took the liberty of changing the Lookup Key property to Lookup Value
Field, and updated variables and unit tests and such. +1 LGTM, built and ran
unit tests, also tried a flow with LookupRecord wi
Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/2123
@MikeThomsen I have reviewed the updated PR. It looks like all of my
concerns are addressed. Thanks for the new iteration! I'm a +1 as long as
@mattyb149 's concerns are all addressed. Thanks!
---
Github user MikeThomsen commented on the issue:
https://github.com/apache/nifi/pull/2123
@markap14 Do you think you could take a look at the updates?
---
Github user MikeThomsen commented on the issue:
https://github.com/apache/nifi/pull/2123
@markap14 Updated based on your feedback and added the ability to merge
either a single field or the entire Mongo result minus the _id field.
---
Github user MikeThomsen commented on the issue:
https://github.com/apache/nifi/pull/2123
@mattyb149 Ok should be good to go now.
---