Daniel (dB.) Doubrovkine

Results 3330 comments of Daniel (dB.) Doubrovkine

Indeed, looks like arrays of fields aren't properly deserializing. I fixed the spec in https://github.com/opensearch-project/opensearch-api-specification/pull/430 and opened https://github.com/opensearch-project/documentation-website/issues/7748 for the docs. ``` curl -k -X POST --json '{"director":"Bennett Miller","title":"Moneyball","year":2011}' -u...

I would start by seeing if the code generator that @Xtansia recently added would just do the right thing and add support for this after https://github.com/opensearch-project/opensearch-api-specification/pull/430 is merged.

I kicked https://github.com/opensearch-project/opensearch-java/actions/runs/10050371147, and it failed, `java.lang.IllegalArgumentException: Multiple types not yet supported: [boolean, null, number, object, string]`. Care to open an issue for this @chanon-onman please? Or maybe fix it?...

@al-niessner looks like the backport failed as expected, I hear @Xtansia is looking into it, but don't hold back if you know/have/want to try a fix before him

I am in full support of a consistent approach @Xtansia. I would potentially preserve the option of using one or the other library for backwards compatibility, hopefully without reflection as...

@Xtansia what are the changes needed in the generator to pick that up? Maybe @dhauverNS can help?

[Catch All Triage - [1](https://github.com/dblock), [2](https://github.com/krisfreedain), [3](https://github.com/amistrn)]

Looks good to me, needs tests, documentation, samples, etc. Take a look at https://github.com/opensearch-project/opensearch-api-specification too. I think we want to derive a lot of this code from spec (@Xtansia is...

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