Skip to content
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

Moving Elastisearch low level client to stable #22107

Merged
merged 1 commit into from
Dec 20, 2021

Conversation

loicmathieu
Copy link
Contributor

Following a discussion on the quarkus-dev mailing list, I move the Elasticsearch low level client to stable.
The high level one remains preview untill we decide what to do with it ...

By the way, on the FAQ https://quarkus.io/faq/#extension-status there is no deprecated status maybe we need to add one.

@loicmathieu
Copy link
Contributor Author

/cc @gsmet @yrodiere

@gsmet
Copy link
Member

gsmet commented Dec 10, 2021

Yeah, we need to decide what to do with this one. I have no idea if we will upgrade it or not and if so what is the future of this.

@yrodiere
Copy link
Member

Yeah, we need to decide what to do with this one. I have no idea if we will upgrade it or not and if so what is the future of this.

By "this one", which one do you mean?

The low-level client is definitely stable and has been open-source forever, with no change in sight: it's still ASL 2.0 in Elasticsearch 7.16 (5 versions after the infamous license changes).

The high-level client, well... it's whole different problem, I agree.

@loicmathieu
Copy link
Contributor Author

For the moment we can upgrade to 7.10.2 but not to a higher one if we don't want to use Elastic licensed client.

I think we should wait to have an extension based on the new client to move past 7.10, I can try to find some hours before the end of the year to work on it, it'll be copy and paste :). Then we can deprecate the high level client extension.
And as Hibernate Search uses the rest client there should not be any issue.

@gsmet gsmet merged commit d9f25a2 into quarkusio:main Dec 20, 2021
@quarkus-bot quarkus-bot bot added this to the 2.7 - main milestone Dec 20, 2021
@loicmathieu loicmathieu deleted the elasticsearch-stable branch December 22, 2021 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants