Github user gglanzani commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2186#discussion_r142768212
  
    --- Diff: 
nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AbstractKiteProcessor.java
 ---
    @@ -47,33 +47,16 @@
     abstract class AbstractKiteProcessor extends AbstractProcessor {
     
         private static final Splitter COMMA = Splitter.on(',').trimResults();
    -    protected static final Validator FILES_EXIST = new Validator() {
    -        @Override
    -        public ValidationResult validate(String subject, String 
configFiles,
    -                ValidationContext context) {
    -            if (configFiles != null && !configFiles.isEmpty()) {
    -                for (String file : COMMA.split(configFiles)) {
    -                    ValidationResult result = 
StandardValidators.FILE_EXISTS_VALIDATOR
    -                            .validate(subject, file, context);
    -                    if (!result.isValid()) {
    -                        return result;
    -                    }
    -                }
    -            }
    -            return new ValidationResult.Builder()
    -                    .subject(subject)
    -                    .input(configFiles)
    -                    .explanation("Files exist")
    -                    .valid(true)
    -                    .build();
    -        }
    -    };
     
         protected static final PropertyDescriptor CONF_XML_FILES
                 = new PropertyDescriptor.Builder()
    -            .name("Hadoop configuration files")
    -            .description("A comma-separated list of Hadoop configuration 
files")
    -            .addValidator(FILES_EXIST)
    +            .name("hadoop-configuration-resources")
    --- End diff --
    
    Good point, fixed


---

Reply via email to