Daniel (dB.) Doubrovkine
Daniel (dB.) Doubrovkine
> > This works to quickly add protobuf, but I think we must refactor the classes to separate transport-specific parts in this one. > > I think I missed adding...
I'll move this to opensearch-devops, hopefully someone will know!
Actually looks like this is coming from reporting ...
So `bulk` expects a line-delimited JSON and the rest of the APIs expect proper JSON. Both are strings, aren't they? Isn't the custom serializer the issue, shouldn't it be producing...
Ok, I think I understand. I agree on `isinstance(data, str)`, want to give it a try?
I think you know as much as I (we) do! ;)
@roshanjrajan-zip we aren't following changes in ES clients, but since those are APLv2 please feel free to port that code in a PR here (make sure to keep their license...
https://github.com/opensearch-project/OpenSearch/pull/10296 is next to be merged, then we can update documentation for 2.13 accordingly.
I'll move this to the ml plugin. Did you get a chance to open a ticket with AWS yet?
Looks good, needs a test please.