Github user mattyb149 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2180#discussion_r165995911
--- Diff:
nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/GetMongo.java
---
@@ -152,21 +146,25 @@ public ValidationResult validate(final String
subject, final String value, final
.displayName("JSON Type")
.name("json-type")
.description("By default, MongoDB's Java driver returns
\"extended JSON\". Some of the features of this variant of JSON" +
- " may cause problems for other JSON parsers that expect only
standard JSON types and conventions. This configuration setting " +
- " controls whether to use extended JSON or provide a clean
view that conforms to standard JSON.")
+ " may cause problems for other JSON parsers that
expect only standard JSON types and conventions. This configuration setting " +
+ " controls whether to use extended JSON or provide a
clean view that conforms to standard JSON.")
.expressionLanguageSupported(false)
.required(true)
.build();
private final static Set<Relationship> relationships;
private final static List<PropertyDescriptor> propertyDescriptors;
+ static final Relationship REL_SUCCESS = new
Relationship.Builder().name("success").description("All files are routed to
success").build();
+
static {
List<PropertyDescriptor> _propertyDescriptors = new ArrayList<>();
_propertyDescriptors.addAll(descriptors);
_propertyDescriptors.add(JSON_TYPE);
_propertyDescriptors.add(USE_PRETTY_PRINTING);
+ _propertyDescriptors.add(CHARSET);
_propertyDescriptors.add(QUERY);
+ _propertyDescriptors.add(QUERY_ATTRIBUTE);
--- End diff --
Since you are adding an (albeit optional) property to GetMongo, there
should be a unit test to cover it.
---
