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
global: add support for ES7 #31
Conversation
.travis.yml
Outdated
- "wget -O - https://download.elasticsearch.org/elasticsearch/release/org/elasticsearch/distribution/tar/elasticsearch/${ES_VERSION}/elasticsearch-${ES_VERSION}.tar.gz | tar xz --directory=/tmp/elasticsearch --strip-components=1" | ||
- "/tmp/elasticsearch/bin/plugin install -b mapper-attachments" | ||
- "wget -O - $ES_URL | tar xz --directory=/tmp/elasticsearch --strip-components=1" | ||
- "if [ ${ES_URL} == ${ES2_DOWNLOAD_URL} ]; then /tmp/elasticsearch/bin/plugin install -b mapper-attachments; fi" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not needed, since full-text file indexing is not part of invenio-record-files
anymore
setup.py
Outdated
@@ -28,6 +28,8 @@ | |||
|
|||
db_version = '>=1.0.0' | |||
|
|||
invenio_search_version = '1.0.0' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be bumped to 1.2.0
setup.py
Outdated
'tests': tests_require, | ||
} | ||
|
||
extras_require['all'] = [] | ||
for name, reqs in extras_require.items(): | ||
if name in ('mysql', 'postgresql', 'sqlite'): | ||
if name in ( | ||
'mysql', 'postgresql', 'sqlite', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These three can be removed as well
tests/test_rest.py
Outdated
|
||
if ES_VERSION[0] == 2: | ||
assert len(options) == 2 | ||
assert {(o['payload']['id'], o['text']) for o in options} == { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, invenio-records-rest
makes sure _source
is always there with these keys (i.e. the same case as below for ES_VERSION[0] > 2
). I think we can keep the _source
version of the tests (since ES2 will be removed soon)
tests/test_rest.py
Outdated
elif ES_VERSION[0] > 2: | ||
assert len(options) == 1 | ||
assert { | ||
(o['_source']['id'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is the same test as above
Co-Authored-By: Glignos <glignos93@gmail.com>
No description provided.