[GitHub] [nifi] jfrazee commented on a change in pull request #5136: NIFI-8668 ConsumeAzureEventHub NiFi processors need to support storag…

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

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

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