[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.


---