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

Upgrade to support ES 5.x #37

Open
lucid-0 opened this issue Dec 20, 2017 · 11 comments
Open

Upgrade to support ES 5.x #37

lucid-0 opened this issue Dec 20, 2017 · 11 comments

Comments

@lucid-0
Copy link
Contributor

lucid-0 commented Dec 20, 2017

I will be working on updating a project that uses collective.elasticsearch to ES version 5+. We've noticed several breaking changes in the es query DSL / API since es 2.4.3 and wonder if there is interest in committing our changes/fixes here.

@fulv
Copy link
Member

fulv commented May 23, 2018

I have several changes I would like to contribute back to upgrade to ES 5+.

I see that there is a test-with-newer-es branch opened by @jensens, but it's mostly isort and travis improvements, no ES-specific fixes.

@jensens, should we keep going on your branch, or do you recommend starting over?

@jensens
Copy link
Member

jensens commented May 24, 2018

Do with it what you want :) or what fits best for you/ your use-case.

We (@agitator and I) decided to go a different path and created https://github.com/collective/collective.es.index
It is a different approach, its not yet perfect. But it works overall very fine on Plone 5.1 and ES 6.

@fulv
Copy link
Member

fulv commented May 26, 2018

Hmm, unfortunately, collective.es.index does not seem to work on Plone 4.

@vangheem
Copy link
Member

I'm open to anyone else taking over maintenance.

@jensens
Copy link
Member

jensens commented May 26, 2018

@fulv no, it was developed for 5.1 only.

@tkimnguyen
Copy link
Member

@jensens what's the difference in approach? Would be good to explain it in your README (or this repo's). @lucid-0 I think this repo is our baby now :)

@fulv
Copy link
Member

fulv commented May 27, 2018

@vangheem I'd be glad to take over maintenance since I'm deep in elasticsearch work and need to make a bunch of upgrades. Thanks for granting me the requisite permissions.

@vangheem
Copy link
Member

What is your pypi username?

@fulv
Copy link
Member

fulv commented May 27, 2018

fulv

@vangheem
Copy link
Member

@fulv added!

@tkimnguyen also added you, Joel and Sam!

@jensens
Copy link
Member

jensens commented May 28, 2018

what's the difference in approach?

It does not index on field base and try to mimic the catalog.
It indexes the whole json serialized content item (we utilize restapis serializers adapters and the in Plone 5.1 here).

The (optional) proxy index provided is a custom query on the ES, one use case is to use it as an drop-in index on SearchableText, but also as anything else and may query anything available.

OTOH, one may query the ES from some view directly and may have really fast listings possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants