opensearch-project / opensearch-learning-to-rank-base

Fork of https://github.com/o19s/elasticsearch-learning-to-rank to work with OpenSearch
Apache License 2.0
14 stars 12 forks source link

[Build] Refactor and Upgrade to OpenSearch 2.10 #23

Closed macohen closed 3 months ago

macohen commented 7 months ago

Is your feature request related to a problem?

Created a new branch for OpenSearch 2.10, but there is refactoring to do to build successfully.

opensearch-learning-to-rank-base % ./gradlew -Dopensearch.version=2.10.0 -Djava.security.manager=allow -Dbuild.snapshot=false build

Removed the stack trace for less noise.

What solution would you like?

Successful build including integ tests.

macohen commented 6 months ago

I've made a bunch of changes to get the plugin to compile on 2.10. Most tests also pass, but I'm not quite sure how to replace toXContent methods since the changes in 2.9 from core. I believe 5 tests are failing. @noCharger I might be able to take a look late this week, but if you feel like giving it a shot, my latest is here: https://github.com/macohen/opensearch-learning-to-rank-base/tree/2.10

jzonthemtn commented 3 months ago

@macohen I see this is building now and there's a release for 2.11.1 now. Does that mean this issue can be closed? If so, I can work on a 2.12.0 version.

macohen commented 3 months ago

Go for it!