-
Notifications
You must be signed in to change notification settings - Fork 267
Switch from elasticsearch-dsl-py dependency to elasticsearch-py (it has been merged into a single package) #491
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
Comments
Thanks a lot @violuke. Need to see the migration guide to port it. |
I've started looking at this in my fork. It's not fully tested yet and needs Elasticsearch v9 too. If I get time for more progress, I'll let you know how it goes, but it's a starting point. The migration should be pretty simple as the code was just copied into the other python package. |
Thanks @violuke ! |
@safwanrahman the migration guide looks very simple https://elasticsearch-dsl.readthedocs.io/en/latest/#elasticsearch-dsl
|
I've made my fork into a PR and allowed maintainer edits in case it's helpful, see #492. |
Sorry for the delay in getting back to you. My branch is working for me on Elasticsearch v9. I'm including it with Poetry like: django-elasticsearch-dsl = { git = "https://github.com/violuke/django-elasticsearch-dsl.git", rev = "7b514692c40a215bac7908e4e348f6f2442fe38f" } Could it please be reviewed and a release compatible with Elasticsreach v9 be released? Thank you 🙏 |
See https://github.com/elastic/elasticsearch-dsl-py/releases/tag/v8.18.0
The text was updated successfully, but these errors were encountered: