ChrisSamo632 edited a comment on pull request #4153: URL: https://github.com/apache/nifi/pull/4153#issuecomment-727070407
> @MikeThomsen it's been a while for me to review NiFi processors.. So please forgive my ignorance . But what did you mean by the "REST API Bundle processors"?? [#4153 (comment)](https://github.com/apache/nifi/pull/4153#issuecomment-603790123) See processors in https://github.com/apache/nifi/tree/main/nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-restapi-processors, which use https://github.com/apache/nifi/tree/main/nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-client-service Think these may need updating too, although a quick look suggests that `_type` is already optional, so maybe all that's needed is to confirm that and ensure there are tests covering the scenario for each processor I raised NIFI-8001 to add the ability to specify dynamic properties for query string parameters on the REST API ES processors (as this is something I believe to be missing but useful) NIFI-8002 also to provide a scroll/pit/search_after processor using the REST API ES approach (although users could create this in a flow already with a combination of existing processors, but would need some relatively complex loop logic); and NIFI-8003 to create an _update_by_query (like the existing _delete_by_query) ---------------------------------------------------------------- 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: [email protected]
