523957 - Admin search returns too much when restricted to a single ID

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Search, defect, P4)

Description

[Wil Clouser [:clouserw]]

Attachment: screenshot

The screenshot explains better than I can here, but when we search for an add-on id surrounded by brackets it should only return that 1 add-on. It's somehow matching others too.  I guess this has to do with the sphinx switch.

Comment 1

[Wil Clouser [:clouserw]]

Attachment: screenshot2

Oh, this might be a bigger problem too

Comment 2

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

Odd... I wonder if by the time you type [220 (4 chars) it's already begun to search.

Should be easy to fix.

Comment 3

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

Um... that second screenshot is alarming...

Comment 4

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

adding this to the 5.3 milestone, it'll be easier to fix while it's fresh in my head.

Comment 5

[Wil Clouser [:clouserw]]

(In reply to comment #4)
> adding this to the 5.3 milestone, it'll be easier to fix while it's fresh in my
> head.

Kicking out of 5.3, but feel free to make a patch while we're in code freeze. :)

Comment 6

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

Attachment: {}

{} {}
  |
\---/

needed a bracket or an else or some piece of code.  I had an else statement laying around, so I just used that.

Comment 7

[Wil Clouser [:clouserw]]

Comment on attachment 414385 [details] [diff] [review]
{}

That does fix the first problem.  I can still get the OOM error searching for [1865]

Comment 8

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

I never got the OOM in the screenshot, so it works for me... I can't imagine why a single query would run out of memory... marking fixed.

------------------------------------------------------------------------
r56827 | [e-mail] | 2009-11-24 16:57:41 -0800 (Tue, 24 Nov 2009) | 2 li

[bug 523957] Added some brackets to make sure we only lookup by addon id if ther

Comment 9

[Vishal Kamdar [:vish_moz]]

addon id 1865 is Adblock Plus 1.1.1 
addon id 4042 is Adblock Plus Filter Uploader 1.7.1 

searching for [4042] gives addon with id 4042 - correct
searching for [1865] gives addon with id 4042 - incorrect

reopening...

Comment 10

[Wil Clouser [:clouserw]]

(In reply to comment #9)
> addon id 1865 is Adblock Plus 1.1.1 
> addon id 4042 is Adblock Plus Filter Uploader 1.7.1 
> 
> searching for [4042] gives addon with id 4042 - correct
> searching for [1865] gives addon with id 4042 - incorrect
> 
> reopening...

That's not what is happening.  You're typing "[1865" which sends an ajax requests and gets back 4042 - this doesn't really matter, since you haven't typed the last "]" yet.  Once you do type that last "]" it sends a second ajax request which just spins because the server is running out of memory (my screenshot above).  It looks like 4042 to you just because the server never returns a different answer so that result is still there.

Comment 11

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

Attachment: []

Using a different method to grab the single addon.

Also displaying the memcache stuff below the screen with the other logs, as not to screw up AJAX requests (Which can supress them) or regular pages (which give warnings for inability to output headers).

Comment 12

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

Deleted branch 523957-oom (was 2aecd08).
host-4-118:~/Projects/Addons/code% git svn log                         (master)
------------------------------------------------------------------------
r57069 | [e-mail] | 2009-11-30 16:13:03 -0800 (Mon, 30 Nov 2009) | 2 li

[bug 523957] Moved some memache notices into the appcontroller log so it can be

Comment 13

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED on https://preview.addons.mozilla.org/en-US/admin/addons