jfrazee commented on a change in pull request #5136:
URL: https://github.com/apache/nifi/pull/5136#discussion_r664874346



##########
File path: 
nifi-commons/nifi-utils/src/main/java/org/apache/nifi/processor/util/StandardValidators.java
##########
@@ -667,6 +667,46 @@ public ValidationResult validate(final String subject, 
final String input, final
         };
     }
 
+    public static Validator createRegexMatchingValidatorWithEL(final Pattern 
pattern, final String validationMessage) {

Review comment:
       @timeabarna I don't think I understand why we need both 
`createRegexMatchingValidator()` and `createRegexMatchingValidatorWithEL()`. If 
`createRegexMatchingValidator()` isn't working with parameters wouldn't it work 
to just make the improvement you've made in the `WithEL` variant?




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


Reply via email to