Daniel (dB.) Doubrovkine
Daniel (dB.) Doubrovkine
I'm leaving this to @msfroh who understands what this code does to decide to merge/not. Thanks for your help @dk2k!
@msfroh So ok to merge as is? Hit merge?
Do you think this is reproducible in a test? Want to try to write one?
I think you narrowed it down to the right question. Maybe @bharath-techie can help us out here?
Thanks for reporting this. Since you have a clear repro, try bisecting this down a change and running tests against a local environment launched via `./gradlew run`?
Moving to docs repo. @SebastianThorn please do feel free to contribute! [Catch All Triage - [1](https://github.com/dblock), [2](https://github.com/refa), [3](https://github.com/andrross), [4](https://github.com/krisfreedain)]
[[Catch All Triage](https://github.com/opensearch-project/.github/blob/main/TRIAGING.md) - [1](https://github.com/dblock), [2](https://github.com/reta), [3](https://github.com/andrross/), [4](https://github.com/krisfreedain)]
[[Catch All Triage](https://github.com/opensearch-project/.github/blob/main/TRIAGING.md) - [1](https://github.com/dblock), [2](https://github.com/reta), [3](https://github.com/andrross/), [4](https://github.com/krisfreedain)]
[[Catch All Triage](https://github.com/opensearch-project/.github/blob/main/TRIAGING.md) - [1](https://github.com/dblock), [2](https://github.com/reta), [3](https://github.com/andrross/), [4](https://github.com/krisfreedain)]
I definitely think a graphql implementation on top of Grape could be cool, even if it's tightly coupled. At @artsy we've been putting node.js middleware in front of multiple API...