spring-data-opensearch icon indicating copy to clipboard operation
spring-data-opensearch copied to clipboard

[FEATURE] Reactive version of the clients

Open ymamakis opened this issue 3 years ago • 11 comments

Is your feature request related to a problem?

Hi there, and apologies if it is too early/it's not on your roadmap/ has already been discussed elsewhere and I have missed it. First of all thanks for all the hard work, I have been eagerly following your work on spring-data-opensearch and already started looking into how this library evolves. We are currently using an older version of Opensearch (1.2.4) alongside the spring-data-elasticsearch library. While playing around with the the merged code, I noticed that there are no equivalents of the reactive versions of the client/configuration. Is this something that you plan to add?

What solution would you like?

Support reactive version of the clients

What alternatives have you considered?

N/A

Do you have any additional context?

N/A

ymamakis avatar Oct 12 '22 15:10 ymamakis

@ymamakis this is a valid feature, worth keeping on roadmap for sure, thank you.

reta avatar Oct 12 '22 16:10 reta

Hello, any news on this subject? We really want to use the opensearch client, but I'm leaning towards moving to the spring data elasticsearch for the reactive connection. Do we have an ETA @wbeckler?

Thanks a lot!

cunhap avatar May 30 '23 09:05 cunhap

Hey @cunhap , afaik there is no work happening at this direction yet

reta avatar May 30 '23 12:05 reta

@cunhap we ended up with using WebClient + AWS request signing

kkocel avatar May 30 '23 13:05 kkocel

Given that it's all Apache 2.0-licensed code, could we just copy the Reactive* clients from https://github.com/spring-projects/spring-data-elasticsearch/tree/main/src/main/java/org/springframework/data/elasticsearch/client/elc and tweak to use the OpenSearch REST high-level client?

msfroh avatar May 21 '24 16:05 msfroh

@msfroh please check https://github.com/opensearch-project/spring-data-opensearch/pull/227, we are getting there

reta avatar May 21 '24 16:05 reta