Open
Bug 1305908
(bmo-fast-keywords)
Opened 8 years ago
Updated 5 years ago
For keywords search, use a table join instead of the weird thing we do now
Categories
(bugzilla.mozilla.org :: Search, defect)
Tracking
()
NEW
People
(Reporter: dylan, Unassigned)
References
Details
Attachments
(1 file)
2.01 KB,
patch
|
dkl
:
review-
|
Details | Diff | Splinter Review |
It seems like we can be a lot faster at keyword searches. Let's give that a try.
Reporter | ||
Comment 1•8 years ago
|
||
there's got to be a downside to this. I also changed quicksearch so that if there's no commas in the bang syntax it does a substring rather than anywords. if you do an exact search this code *flies*
Attachment #8795593 -
Flags: review?(dkl)
Comment 2•8 years ago
|
||
Comment on attachment 8795593 [details] [diff] [review] 1305908_1.patch Review of attachment 8795593 [details] [diff] [review]: ----------------------------------------------------------------- Needs to handle the negative conditions such as nowords, nowordssubstr, noequals, etc. See Bugzilla::Search::_multiselect_negative dkl ::: Bugzilla/Search.pm @@ +2649,5 @@ > > +sub _keywords_nonchanged { > + my ($self, $args) = @_; > + my ($chart_id, $joins, $fields, $operator, $value) = > + @$args{qw(chart_id joins fields operator value)}; my ($chart_id, $joins) = @$args{qw(chart_id joins)};
Attachment #8795593 -
Flags: review?(dkl) → review-
Reporter | ||
Updated•8 years ago
|
Alias: bmo-fast-keywords
Reporter | ||
Updated•5 years ago
|
Assignee: dylan → nobody
You need to log in
before you can comment on or make changes to this bug.
Description
•