jfrazee commented on a change in pull request #5392:
URL: https://github.com/apache/nifi/pull/5392#discussion_r710394742
##########
File path:
nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/PutMongo.java
##########
@@ -288,17 +288,28 @@ protected WriteConcern getWriteConcern(final
ProcessContext context) {
writeConcern = WriteConcern.UNACKNOWLEDGED;
break;
case WRITE_CONCERN_FSYNCED:
- writeConcern = WriteConcern.FSYNCED;
+ writeConcern = WriteConcern.JOURNALED;
+ getLogger().warn("Using deprecated write concern FSYNCED");
Review comment:
`getWriteConcern()` gets called in `onTrigger()` so I think this is
going to be really chatty. It might be helpful to just do this warning in the
`@onScheduled` in the abstract class. It's not using EL so it doesn't even need
to be called in `onTrigger()`.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
