search should ngram index charm and bundle names for search
Bug #1314239 reported by
Richard Harding
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| charmworld |
Fix Released
|
High
|
Brad Crittenden | ||
Bug Description
This is to enable search to find the sugarcrm charm with the following searches
- sugar
- crm
Related branches
lp:~reedobrien/charmworld/ngrams
- Juju Gui Bot: Approve (continuous-integration)
- Richard Harding: Approve
- j.c.sackett (community): Approve
-
Diff: 359 lines (+184/-21)3 files modifiedMakefile (+2/-2)
charmworld/search.py (+104/-8)
charmworld/tests/test_search.py (+78/-11)
lp:~reedobrien/charmworld/es-migration
- Juju Gui Bot: Approve (continuous-integration)
- Brad Crittenden (community): Approve
-
Diff: 370 lines (+281/-41)5 files modifiedcharmworld/migrations/versions/027_use_ngrams.py (+35/-0)
charmworld/migrations/versions/tests/__init__.py (+1/-0)
charmworld/migrations/versions/tests/data/migration_027.py (+166/-0)
charmworld/migrations/versions/tests/test_migrations.py (+79/-4)
charmworld/search.py (+0/-37)
| Changed in charmworld: | |
| assignee: | nobody → Brad Crittenden (bac) |
| status: | Triaged → In Progress |
Revision history for this message
| Reed O'Brien (reedobrien) wrote | #1 |
Revision history for this message
| Juju Gui Bot (juju-gui-bot) wrote | #2 |
| Changed in charmworld: | |
| status: | In Progress → Fix Committed |
| Changed in charmworld: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This is started in http://
- validation that other search calls work
- and tests fixed which broke
- add a new one for trigrams.
It appears to work at /api/3/
Working with Brad to validate assumptions and ensure it doesn't break GUI expectations.