Daniel (dB.) Doubrovkine

Results 3350 comments of Daniel (dB.) Doubrovkine

[Catch All Triage - [1](https://github.com/andrross), [2](https://github.com/dblock), [3](https://github.com/AmiStrn), [4](https://github.com/reta), [5](https://github.com/finnegancarroll)]

Thanks @AntonEliatra. Maybe try to write a [YAML REST test](https://github.com/opensearch-project/OpenSearch/blob/main/TESTING.md#testing-the-rest-layer) for this one short of a fix? We would also love a test and any missing support for analysis filters...

[Catch All Triage - [1](https://github.com/dblock), [2](https://github.com/andrross), [3](https://github.com/sandervandegeijn), [4](https://github.com/AmiStrn), [5](https://github.com/udabhas)]

> @msfroh @reta Would appreciate a review, will open a docs PR to reflect the new parameter change. And https://github.com/opensearch-project/opensearch-api-specification please.

Good idea. In the case where the diff is a 1-liner, try to apply both by default.

[Catch All Triage - [1](https://github.com/andrross), [2](https://github.com/dblock), [3](https://github.com/AmiStrn), [4](https://github.com/reta), [5](https://github.com/finnegancarroll)]

@pepper-kim Thanks for the bug report. It could be a regression, care to help by writing a [YAML REST test](https://github.com/opensearch-project/OpenSearch/blob/main/TESTING.md#testing-the-rest-layer) that reproduces the bug and maybe trying to bisect it...

> For client (Java, Go, Python etc), would have support to optionally use new protobuf-based server API with minimal changes (i.e. no need to rewrite an application already using the...

Re: benefits I expect grpc + protobuf to improve both performance and throughput over HTTP/2 JSON. You're right to call this out though, @amberzsy were your benchmarks using HTTP/2?