[GitHub] nifi issue #2586: NIFI-5021 Moving nifi-elasticsearch-client-service-api to ...

2018-04-02 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/2586 Agreed, will merge ---

[GitHub] nifi issue #2586: NIFI-5021 Moving nifi-elasticsearch-client-service-api to ...

2018-04-02 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2586 I think that would be a good idea. ---

[GitHub] nifi issue #2586: NIFI-5021 Moving nifi-elasticsearch-client-service-api to ...

2018-04-01 Thread JPercivall
Github user JPercivall commented on the issue: https://github.com/apache/nifi/pull/2586 @bbende @MikeThomsen since there is another RC of 1.6.0, did you want to merge this in? ---

[GitHub] nifi issue #2586: NIFI-5021 Moving nifi-elasticsearch-client-service-api to ...

2018-03-28 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2586 Sounds good to me. ---

[GitHub] nifi issue #2586: NIFI-5021 Moving nifi-elasticsearch-client-service-api to ...

2018-03-28 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/2586 Thanks for reviewing, I'm going to wait til 1.6.0 release finalizes before merging this because there will likely be conflicts when the poms get updated to 1.7.0-SNAPSHOT. ---

[GitHub] nifi issue #2586: NIFI-5021 Moving nifi-elasticsearch-client-service-api to ...

2018-03-27 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2586 +1 LGTM. I ran it and didn't notice any regression. ---