exceptionfactory commented on code in PR #4044:
URL: https://github.com/apache/nifi/pull/4044#discussion_r1194307760


##########
nifi-nar-bundles/nifi-jolt-record-bundle/nifi-jolt-record-processors/src/main/java/org/apache/nifi/processors/jolt/record/JoltTransformRecord.java:
##########
@@ -141,12 +145,23 @@ public class JoltTransformRecord extends 
AbstractProcessor {
     static final PropertyDescriptor JOLT_SPEC = new 
PropertyDescriptor.Builder()
             .name("jolt-record-spec")
             .displayName("Jolt Specification")
-            .description("Jolt Specification for transform of record data. 
This value is ignored if the Jolt Sort Transformation is selected.")
+            .description("Jolt Specification for transform of record data. 
Only one of 'Jolt Specification' or 'Path To Jolt Specification' may be used. "
+                    + "This value is ignored if the Jolt Sort Transformation 
is selected.")
             
.expressionLanguageSupported(ExpressionLanguageScope.FLOWFILE_ATTRIBUTES)
             .addValidator(StandardValidators.NON_EMPTY_VALIDATOR)
             .required(false)
             .build();
 
+    static final PropertyDescriptor JOLT_SPEC_FILE = new 
PropertyDescriptor.Builder()
+            .name("jolt-spec-file")
+            .displayName("Path To Jolt Specification")
+            .description("Path to location of a JOLT specification file. Only 
one of 'Jolt Specification' or 'Path To Jolt Specification' may be used. "
+                    + "This value is ignored if the Jolt Sort Transformation 
is selected.")
+            .required(false)
+            .identifiesExternalResource(ResourceCardinality.SINGLE, 
ResourceType.FILE)

Review Comment:
   @mattyb149 Is it possible to enhance the existing `Jolt Specification` 
property to support multiple resource types? That way the existing property 
could be either a text string as it is now, or a file reference.



-- 
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]

Reply via email to