Github user zenfenan commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2776#discussion_r194246883
  
    --- Diff: 
nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/AbstractMongoProcessor.java
 ---
    @@ -275,4 +303,13 @@ protected void writeBatch(String payload, FlowFile 
parent, ProcessContext contex
             session.getProvenanceReporter().receive(flowFile, getURI(context));
             session.transfer(flowFile, rel);
         }
    +
    +    protected void configureMapper(String setting) {
    --- End diff --
    
    @MikeThomsen I again went through the link @joewitt shared. Anyway, we are 
configuring `objectMapper` before using it, at least in this case. That 
guarantees thread safety. https://stackoverflow.com/a/3909846/2973446. Anyway I 
have marked it synchronized. Let me know, if there are any concerns.
    
    @joewitt If you see something odd with this one, please share it over here.


---

Reply via email to