MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-553424913
@mattyb149 I think we're good to go now.
This is an auto
MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-552912581
@mattyb149 changes are all in now and Travis is passing.
MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-535028790
Also, once this is done, I'll probably pivot to refactoring where needed to
add a bulk delete by id processor and whateve
MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-535028229
@mattyb149 I refactored the heck out of this one because I decided that what
we really need is just a smooth put record p
MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-508883974
@mattyb149 gave it some though, and I agree that schema inference makes the
Json processor OBE at this point. I'll rip it
MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-508782846
@mattyb149 I think I get can get these done today. Broadly, my purpose in
creating these was to give us a clean path forw
MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-506443901
@mattyb149 I think I figured it out. Rebased and force pushed. Didn't see
anything missing locally and the integration te
MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-506304976
@mattyb149 I used the github conflict resolver to get it building. Tried
rebasing locally and kept running into an issue
MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-505981796
@mattyb149 Sure. Appreciate it. Might be a little busy today, but I'll try
to get it done today.
---
MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-496018185
@mattyb149 Added error handling taken from @JPercivall 's commit for
`PutElasticsearchHttpRecord`.
-
MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-489455878
@mattyb149 @zenfenan @JPercivall could we restart the review and close this
out?
---
MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-485247410
@AaronLeon They did introduce some breaking changes to their REST API in
7.X. Specifically in the update API. 7.X:
MikeThomsen commented on issue #2861: NIFI-5248 Added new Elasticsearch json
and record processors.
URL: https://github.com/apache/nifi/pull/2861#issuecomment-483655602
Now that we have Elastic 7.0.0 to contend with, I think the issue has been
forced on using the low-level client at least
13 matches
Mail list logo