Github user MikeThomsen commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2778#discussion_r194241017
--- Diff:
nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/PutMongoRecord.java
---
@@ -131,7 +131,7 @@ public void onTrigger(final ProcessContext context,
final ProcessSession session
for (String name : schema.getFieldNames()) {
--- End diff --
Mainly preference. When went back and looked at it, a single recursive
function seemed like the right call. I still think it is. Not much to be saved
by mixing it into that loop IMO.
---
