SavtechSolutions commented on a change in pull request #3227: NIFI-5909 PutElasticsearchHttpRecord doesn't allow to customize the timestamp format URL: https://github.com/apache/nifi/pull/3227#discussion_r246490098
########## File path: nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-processors/src/main/java/org/apache/nifi/processors/elasticsearch/PutElasticsearchHttpRecord.java ########## @@ -178,6 +178,36 @@ .required(true) .build(); + static final PropertyDescriptor DATE_FORMAT = new PropertyDescriptor.Builder() + .name("put-es-record-date-format") + .displayName("Date Format") + .description("Custom date format to use when converting fields of date type " + Review comment: Also, I see the existing PDs using the SimpleDateFormatValidator to validate formats. It probably makes sense for me to use the same, but this means an additional POM dependency to nifi-standard-record-utils - is it OK to add that? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services