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

Reply via email to