-
Notifications
You must be signed in to change notification settings - Fork 1.8k
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
cubejs-elasticsearch-driver not compatible with v7.x #3520
Comments
If you are interested in working on this issue, please leave a comment below and we will be happy to assign the issue to you. |
By further investigating it looks like at line 134 tries to cal getMapping on the index '.geoip_databases', which causes the exception. In version 6.8 this caused an error, but the |
I came up with this fix If you think it's ok, I will open a PR |
Hey @gabri94 ! PR would be very much appreciated! |
Any chance of getting this merged in? @gabri94 fix seems to work with Elastic Cloud which is running on 7.16 |
@stueccles @gabri94 We just need a PR here. |
Any update on the PR @gabri94 ? |
The fix is only partially working, so It doesn't make sense to merge it. |
issue still persist (trying with ElasticSearch 7.15) |
i had made changes as present in @gabri94 's PR and also added a small tweak in one of the line as shown below and it worked for me in version 7.14.1
|
See cube-js#3520 for more information on these changes
any updates on this? |
I have similar issue on latest elasticsearch: 8.6.2 |
Describe the bug
The backend package works with elasticsearch up to v6.8. With v7.x it causes the following error:
I've tested this behaviour both with elastic.io and a local vanilla elasticsearch v7.15 image.
On the other hand, the library works correctly with elasticsearch v.6.8.18
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The lib should work with the latest version of elasticsearch
The text was updated successfully, but these errors were encountered: