[GitHub] [nifi] MikeThomsen commented on a change in pull request #5304: NIFI-9041 Replaced JUnit 4 only testing configuration with a combinat…

[GitBox <gi...@apache.org>]

MikeThomsen commented on a change in pull request #5304:
URL: https://github.com/apache/nifi/pull/5304#discussion_r687732448



##########
File path: nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-services/src/main/java/org/apache/nifi/mongodb/MongoDBLookupService.java
##########
@@ -67,6 +66,10 @@
     "then the entire MongoDB result document minus the _id field will be returned as a record."
 )
 public class MongoDBLookupService extends JsonInferenceSchemaRegistryService implements LookupService<Object> {
+    public static final PropertyDescriptor LOCAL_SCHEMA_NAME = new PropertyDescriptor.Builder()
+            .fromPropertyDescriptor(SCHEMA_NAME)
+            .expressionLanguageSupported(ExpressionLanguageScope.VARIABLE_REGISTRY)
+            .build();

Review comment:
       Yeah. When expressionLanguageSupported was refactored to use an enum, it was changed to be for FLOWFILE_VARIABLE and not VARIABLE_REGISTRY which is the only option a controller service can reasonably accept.




-- 
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-unsubscribe@nifi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org