ijokarumawak commented on issue #3556: NIFI-6403: Fixed 
FIELD_INCLUDE_QUERY_PARAM value for Elastic 6.6+
URL: https://github.com/apache/nifi/pull/3556#issuecomment-574560220
 
 
   I think we need to handle different Elasticsearch major versions better. 
Currently we only have `nifi-elasticsearch-5-nar` and `nifi-elasticsearch-nar`. 
If we are going to maintain `nifi-elasticsearch-nar` for the latest 
Elasticsearch version, we probably have to add `nifi-elasticsearch-6-nar`.
   
   That is probably what @MikeThomsen tried to implement with #2782 . Then I'd 
finish review/merge #2782 first. We also have to add documentation what 
Elasticsearch version `nifi-elasticsearch-nar` is designed to work.
   
   @MikeThomsen do you agree with that? If so, I'll resume reviewing #2782.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to