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

Updates #44

Merged
merged 4 commits into from
Mar 7, 2018
Merged

Updates #44

merged 4 commits into from
Mar 7, 2018

Conversation

jciolek
Copy link
Collaborator

@jciolek jciolek commented Feb 26, 2018

Summary

General maintenance, including preparation for Elasticsearch 6.x support:

  • Added missing tests for $nested and $sqs.
  • Added tests for Elasticsearch 5.6
  • Moved db setup and teardown to a separate file - it's the first step towards supporting Elasticsearch 6.0 in the tests. The test-db.js file will be dealing with schema differences between Elasticsearch 6.x and 5.x.
  • Other minor cosmetic fixes.

Also, upgrade elasticsearch client to 14.1.0 and api version
to the actual elasticsearch version used in the test app.
This is to prepare for serving slightly different schema
for Es 6.0. There were some breaking changes, e.g. around
parent-child relationship. Moving schema management to a separate file
should allow the tests to stay focused and not care much
about particularites of the db config.
@jciolek jciolek requested a review from daffl February 26, 2018 20:00
Copy link
Member

@daffl daffl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. :shipit:

@salamynder
Copy link

When will this be merged? I'm keen on ES6 support (only one type per index, routing-property mandatory for parent/child). I can try to contribute, but first this thingy here should be integrated?

@daffl daffl merged commit 9841aee into feathersjs-ecosystem:master Mar 7, 2018
@jciolek
Copy link
Collaborator Author

jciolek commented Mar 7, 2018

Hey @salamynder, I'm actively working on Elasticsearch 6.x support. There is another PR #45 for general maintenance and after that there will be one with Elasticsearch 6.x.

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

Successfully merging this pull request may close these issues.

3 participants