querqy / querqy-elasticsearch

Querqy for Elasticsearch
Apache License 2.0
45 stars 19 forks source link

Any plan to bump for elasticSearch 8.15 (or as close as possible)? #127

Open smulder opened 1 month ago

smulder commented 1 month ago

Not entirely sure what would need to be done to get this running for 8.15 or 8.14.2, but I see a pull request for 8.14.2 already that didn't build and closed (#126). Any chance of getting more current?

Thanks tons! Scott

renekrie commented 1 month ago

Hi @smulder, The plan is definitely there but if anybody could lend a hand it would be great. The build for 8.14.2. broke because an import statement failed. Oftentimes, it means that a class just moved packages in ES or that it was renamed. Feel free to have a look at it and create a PR (I might have some time end of next week only).

smulder commented 1 month ago

I am unfortunately not a Java developer (always fun to learn something new though), but I can poke around to see if anything universally understandable jumps out at me and report back with anything I think may be helpful.

smulder commented 1 month ago

@renekrie, I submitted a pull request with some alterations that may or may not be helpful. Being that this is the absolute first time I have touched java or a java IDE, I am sure there are some things you will laugh at. Just let me know if it in any way helps, if you can take it from there or if you would like to guide me to do anything further. Please be honest if it is total crap also! Thanks in advance.

drieschel commented 3 weeks ago

Hello there, I just created PR https://github.com/querqy/querqy-elasticsearch/pull/129, which aims to add compatiblity to Elasticsearch 8.15.0.