Oliver Eilhard comments

Results 116 comments of


                                            Oliver Eilhard

go get adds github.com/olivere/elastic/v7 as an indirect dependency in go.mod

Thanks for reporting. Yep, there's something wrong. I probably know why this happens, and will fix in the next release.

go get adds github.com/olivere/elastic/v7 as an indirect dependency in go.mod

Hmm... for the moment, I'm not sure where this is coming from. Need to investigate a bit more.

Reduce memory usage on bigger search hits result

Which amount of memory usage would be acceptable for your application? That's 1.4 MB—not too uncommon these days.

I get elastic: Error 400 (Bad Request) with some slice ids in sliced scroll call

Can you provide a reproducible test case that yields the error? I'm not really sure this is a driver issue.

NewMatchPhraseQuery isn't retrieving document

Can you enable the trace log to see the communication between elastic and ES. That’s what I typically do for debugging these issues. See https://github.com/olivere/elastic/wiki/Logging

Client fails to handle context cancellation correctly

Thanks. The elastic.v5 branch is no longer maintained, but we can port back the changes to v6 and v7. It looks like this is basically what we have [here](https://github.com/olivere/elastic/blob/release-branch.v7/errors.go#L95-L109), right?...

For one retry, length of list of fixed intervals passed to NewSimpleBackoff should be 2

Can you make a test similar to [this one](https://github.com/olivere/elastic/blob/release-branch.v7/backoff_test.go#L36-L79) that fails?

For one retry, length of list of fixed intervals passed to NewSimpleBackoff should be 2

I agree, that seems to be wrong. It should be `-1` by default, or incremented after retrying.

For one retry, length of list of fixed intervals passed to NewSimpleBackoff should be 2

This will need to be backported to 6.x and 5.x.

bulk: Add easyjson bindings

Thanks for your commit. I probably won't add anything like this to the core library. Last time I checked the cost of deserialization was neglectable compared to the I/O and...