-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[META] Streaming Indexing API #9065
Comments
@reta pretty great stuff - there's other work on improving client/server performance, how do you see us fit work on Protobuf (@VachaShah) / gRPC? |
Thanks @dblock, the gRPC would definitely benefit from the reactive streaming part. From other side, I assume the gRPC would be used as node transport layer (at least, initially), so the HTTP reactive layer (as suggested alternative transport for HTTP clients) would benefit enormously from that - we will be having end-to-end reactive processing pipeline. |
@reta Looking to collaborate on the Streaming API changes and see by when it can make it to OpenSearch release. |
@shwetathareja that would be great, the first thing is to get this one in #9672 - the pull request adds new HTTP transport based on Reactor Netty 4 with streaming support, it is well on schedule for Once the transport is there, we could split the work, there are quite a few opportunities for doing that in parallel, thank you. |
Hi, I'm really interested in testing this feature. Does the current implementation support bi-directional streaming, I.e. returning responses for each chunk/document? Currently I'm streaming the request, but OpenSearch appears to wait until the request is complete before sending the response. Not sure whether my setup is wrong, or if this is expected. Thanks Edit: this works as expected. My code was the issue! |
Just for visibility, yes the implementation support bi-directional streaming, thanks @T-J-L ! |
I didn't want to clutter this issue so created a separate one here, but any help would be appreciated 😄 |
Is your feature request related to a problem? Please describe.
The meta issue to track the Streaming Indexing API progress
Describe the solution you'd like
As outcome of the #5001 & #7273, we have outlined the way such steaming support could be integrated into OpenSearch.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: