Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Disabling the fulltext option for the repository model for now #223

Merged
merged 2 commits into from
Jul 10, 2015

Conversation

mssola
Copy link
Collaborator

@mssola mssola commented Jul 10, 2015

This works around a regression in the search feature. Basically, MariaDB seems
to fail at some searches when the fulltext option has been enabled. We are
still investigating about this.

This works around a regression in the search feature. Basically, MariaDB seems
to fail at some searches when the `fulltext` option has been enabled. We are
still investigating about this.
@flavio
Copy link
Member

flavio commented Jul 10, 2015

LGTM, merge as soon as all the tests are green

@mssola
Copy link
Collaborator Author

mssola commented Jul 10, 2015

Note that the test on the spec/controllers/search_controller_spec.rb file has been reduced. This is because the contents of the @repositories is already being tested by the spec/features/search_feature_spec.rb file. The tests there were ok because we are passing js: true which truncates the DB and fixes the indexes issues.

mssola added a commit that referenced this pull request Jul 10, 2015
Disabling the `fulltext` option for the repository model for now
@mssola mssola merged commit 02911b6 into SUSE:master Jul 10, 2015
@mssola mssola deleted the search branch July 10, 2015 16:16
@mssola mssola mentioned this pull request Nov 2, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants