[GitHub] [nifi] ChrisSamo632 commented on pull request #6903: NIFI-11111 add option to output Elasticsearch error responses as FlowFile to PutElasticsearchJson and PutElasticsearchRecord

2023-04-24 Thread via GitHub
ChrisSamo632 commented on PR #6903: URL: https://github.com/apache/nifi/pull/6903#issuecomment-1520887667 [NIFI-11480](https://issues.apache.org/jira/browse/NIFI-11480) for the above ☝️ discussion), FYI @davis-anthony -- This is an automated message from the Apache Git Service. To

[GitHub] [nifi] ChrisSamo632 commented on pull request #6903: NIFI-11111 add option to output Elasticsearch error responses as FlowFile to PutElasticsearchJson and PutElasticsearchRecord

2023-04-21 Thread via GitHub
ChrisSamo632 commented on PR #6903: URL: https://github.com/apache/nifi/pull/6903#issuecomment-1517903668 @davis-anthony that would definitely warrant a new Jira ticket to make such a change I think. Some sort of "half-way house" approach may be more appropriate, e.g. like

[GitHub] [nifi] ChrisSamo632 commented on pull request #6903: NIFI-11111 add option to output Elasticsearch error responses as FlowFile to PutElasticsearchJson and PutElasticsearchRecord

2023-04-19 Thread via GitHub
ChrisSamo632 commented on PR #6903: URL: https://github.com/apache/nifi/pull/6903#issuecomment-1514554132 @davis-anthony this PR if/when Approved (@MikeThomsen / @mattyb149 ) was not going to introduce such behaviour, *but* it *does* make sense for `PutElasticsearchJson` so I've added a

[GitHub] [nifi] ChrisSamo632 commented on pull request #6903: NIFI-11111 add option to output Elasticsearch error responses as FlowFile to PutElasticsearchJson and PutElasticsearchRecord

2023-03-20 Thread via GitHub
ChrisSamo632 commented on PR #6903: URL: https://github.com/apache/nifi/pull/6903#issuecomment-1476912051 @mattyb149 I believe I addressed your comments @MikeThomsen do the updates help with clarity around the transfer of FlowFiles/assignment of attributes? -- This is an automated