-
Notifications
You must be signed in to change notification settings - Fork 8
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
286 switch from solr to elastic search #1090
Conversation
end | ||
|
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.
Style/EmptyLinesAroundBody: Extra empty line detected at body end.
please change the travis.yml file according to the changes that elasticsearch instead of solr will be started |
include Elasticsearch::Model::Callbacks | ||
|
||
mapping do | ||
indexes :name, type: 'string', :analyzer => 'snowball' |
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.
Style/HashSyntax: Use the new Ruby 1.9 hash syntax.
d5d67e3
to
56af861
Compare
|
||
if in_repository? | ||
respond_with OntologySearch.new.make_repository_keyword_list_json(repository, prefix) | ||
def search |
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.
Metrics/CyclomaticComplexity: Cyclomatic complexity for search is too high. [7/6]
Metrics/MethodLength: Method has too many lines. [22/10]
Metrics/PerceivedComplexity: Perceived complexity for search is too high. [8/7]
Present also checks for nil, empty only checks for emptiness.
If there's a change in an ontology, the indexing will be updated.
5a474ed
to
8d103e2
Compare
Fix port for testing elasticsearch.
6cc7819
to
72de428
Compare
But on our local machines we'll use 9250 and the test-cluster.
72de428
to
daa05c1
Compare
👍 |
…_search 286 switch from solr to elastic search
Should do the first fixes for the search with elasticsearch instead of solr