1039969 - Multiword search

Status: RESOLVED FIXED

(Socorro Graveyard :: Middleware, defect)

Description

[David Teller [:Yoric] - still alive but not very active]

Using SuperSearch, I can search for all crashes in which field `async_shutdown_timeout` does not contain "experiments.jsm" (for some reason, I need to lowercase it), but if I search for all crashes in which the field does not contain "experiments.jsm shutdown", this fails to filter out anything.

The same issue appears when I search for all crashes in which a field contains "experiments.jsm shutdown", as this returns no crash.

Comment 1

[David Teller [:Yoric] - still alive but not very active]

Adrian, is this a well-known behavior? Is there a workaround?

Comment 2

[[DEACTIVATED] Adrian Gaudebert]

Can you please provide some URLs? Depending on how you use the API, it could be an expected behavior. For example, if you say ``async_shutdown_timeout=!~experiments.jsm%20shutdown`` that becomes a "phrase match", ie it looks from the phrase "experiments.jsm shutdown". The solution would be to use ``async_shutdown_timeout=!~experiments.jsm&async_shutdown_timeout=!~shutdown`` (that should work, if it doesn't then it's a bug).

Comment 3

[David Teller [:Yoric] - still alive but not very active]

Well, "phrase match" sounds good. The following query should return all the async_shutdown_timeout crashes that do not contain "experiments.jsm shutdown", shouldn't it?

https://crash-stats.mozilla.com/api/SuperSearch/?async_shutdown_timeout=!~experiments.jsm+shutdown&async_shutdown_timeout=!__null__

(this is the query built from the webapp) However, it returns crashes that contain "Experiments.jsm shutdown". I have also tried variants with `%20` instead of `+`, quotes, etc. it doesn't change anything.

Note that it works if I just use "experiments.jsm", rather than "experiments.jsm shutdown". Not if I use "Experiments.jsm", though.

Comment 4

[David Teller [:Yoric] - still alive but not very active]

Any news about this bug? This is blocking investigations on possible topcrashers.

Comment 5

[[DEACTIVATED] Adrian Gaudebert]

Pull request: https://github.com/mozilla/socorro/pull/2310

Comment 6

[[github robot]]

Commits pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/f802be05d64f535307db53fc3c0ca3ea52393707
Fixes bug 1039969 - SuperSearch API now considers all arguments as lists.

https://github.com/mozilla/socorro/commit/ccdf7aa171791457fc2ab3794da4ec81bb28fadb
Merge pull request #2310 from AdrianGaudebert/1039969-api-supersearch-fields-are-lists

Fixes bug 1039969 - SuperSearch API now considers all arguments as lists.