exceptionfactory commented on code in PR #6460: URL: https://github.com/apache/nifi/pull/6460#discussion_r1003428262
########## nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/gridfs/PutGridFS.java: ########## @@ -212,9 +267,10 @@ private boolean canUploadFile(ProcessContext context, FlowFile input, String buc Document query; if (uniqueness.equals(UNIQUE_BOTH.getValue())) { - query = new Document().append("filename", fileName).append("md5", hash); + query = new Document().append("filename", fileName) + .append("metadata", new Document(hashMetadataName, hash)); } else if (uniqueness.equals(UNIQUE_HASH.getValue())) { - query = new Document().append("md5", hash); + query = new Document().append("metadata", new Document(hashMetadataName, hash)); Review Comment: Thanks for the reply @MikeThomsen. In order to avoid breaking existing users of PutGridFS, it seems like we either need to add server version checking, or perhaps a new configurable strategy property along the lines that you mentioned. Either way, we should have a path forward that maintains the current functionality, but also provides a new option that moves away from the deprecated `md5` approach. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org