=== wedgwood is now known as wedgwood_away
StevenKThe bugtask subscriber vocab is built in place00:07
StevenKwgrant: mawson is updated, my patch cowboyed and JS unshelved, but it only renders timeouts01:03
StevenKAh, /auditor finally works01:04
StevenKlifeless: Ah, the auditor project page on DF01:05
StevenKwgrant: Right, so the exact matches are great for DF01:20
StevenKVery fast01:20
wgrantStevenK: I'd certainly hope so!01:20
StevenKMaybe we want to restrict the MP vocab01:20
wgrantOne thing I thought of is that you probably want to do a case-insensitive match01:21
wgrantOn Branch.name01:21
StevenKwgrant: So searching for 'devel' on proposing a LP branch returns too many matches01:21
StevenKBut I'd honestly expect to usually paste in a URL01:21
wgrantStevenK: You're conditioned to paste a URL because search is so useless today01:22
wgrantIt usually times out, and when it doesn't time out it doesn't return useful results unless you're very accurate01:22
StevenKMaybe we want to search on unique_name too01:23
StevenKBut I discounted that01:23
wgrantPrefix search on unique_name makes sense if there's a / in the term01:24
wgrantOtherwise I don't think it's valuable01:24
StevenKwgrant: Thoughts on using BranchRestrictedOnProduct for BMP's vocab?01:27
wgrantStevenK: Remember that BMPs work for more than products01:28
StevenKwgrant: Prefix search? Then searching for launchpad/devel won't work01:28
wgrantStevenK: That's true.01:30
wgrantBut mmm01:30
StevenKcontains_string is probably too heavy handed for unique_name01:30
wgrantYes, but it's all we have without schema changes01:31
wgrantAnd I don't think it's going to be toooo long for a trigram to be effective01:34
wgrantBut you should try.01:34
StevenKYou mean like http://pastebin.ubuntu.com/1641700/ :-)01:36
wgrantStevenK: Right, that sort of thing01:37
wgrantIdeally a unique_name match would be component-wise rather than character-wise, but we can't really do that atm01:37
StevenKLet me patch that onto DF01:38
StevenKwgrant: Oh, shall we TGRM up name and unique_name ?01:38
wgrantStevenK: Try with and without01:39
wgrantMax length of unique_name today is 14101:39
wgrantWhich isn't great, but not fatal01:39
StevenKWhat's that one?01:40
wgrant ~openerp-commiter/openobject-addons/trunk-configuration_wizard_improvement-atp-improve-general-apply-button-color-in-configuration-wizard-pja |         14101:40
StevenKThat's digusting01:40
wgrant ~78luphr0rnk2nuqimstywepozxn9kl19tqh0tx66b5dki1xxsh5m-launchpad-a811i2i3ytqlsztthjth0svbccw8inm65tmkqp9sarr553jq53in/syncany/delimiters       |         13501:41
wgrant ~78luphr0rnk2nuqimstywepozxn9kl19tqh0tx66b5dki1xxs-phn5hho65-a811i2i3ytqlsztthjth0svbccw8inm65tmkqp9sarr553jq5/sysbench/sysbench-pg           |         13101:41
wgrantThen more openerp01:41
StevenKI can't complain about the 135 or 131, those are effectively our fault01:41
StevenKApparently, the searches are all ~ 2 seconds01:45
wgrantie. awful01:46
wgrantget a plan01:46
StevenKwgrant: http://pastebin.ubuntu.com/1641721/01:49
wgrantStevenK: Heh01:49
wgrantStevenK: FYI I prefer to name cowboyed indices and tables temp_* so they can be easily found and removed02:02
StevenKIt's still 440 ms with the TRGM02:06
StevenKwgrant: Would gist rather than gin help?02:36
wgrantStevenK: No02:36
wgrantGiST vs GIN is mostly a write vs read tradeoff02:36
wgrantGIN is slower at writes, but faster at reads02:36
wgrantSo we want GIN02:36
StevenKBut then we're stuffed even with the index02:38
wgrantNot if it's a relatively consistent 4s02:38
wgrantIt's not create02:38
wgrantBut it's better than 2s consistent.02:38
wgrantAnd will be faster on prod02:38
wgrantStevenK: Is this unique_name or name?02:38
StevenKThe former02:39
wgrantWhich is rare02:39
wgrantthough most of the length will be in name too, see how it performs02:39
StevenKThat index is just creating now02:39
StevenKTRGM for name is 300ms02:42
StevenKI was expecting more on the order of 10002:43
StevenKNot 300 or 40002:43
wgrantNot with a long large GIN index02:43
StevenKWhat's the \d magic to see the index size?02:44
StevenKHoly crap, they're 200MiB all up02:45
StevenKwgrant: So, push the changes for this branch, and create a new branch for two shiny TRGMs?02:47
wgrantI think so02:48
* StevenK stabs the branch scanner and reaches for carob03:14
StevenKwgrant: https://code.launchpad.net/~stevenk/launchpad/branch-trgm-indicies/+merge/14808203:33
StevenKwgrant: And new-branch-search is updated03:33
=== almaisan-away is now known as al-maisan
=== dimitern is now known as dimitern_afk
adeuringgood morning08:59
=== al-maisan is now known as almaisan-away
=== dimitern_afk is now known as dimitern
=== BradCrittenden is now known as bac
=== wedgwood_away is now known as wedgwood
=== teknico_ is now known as teknico
=== almaisan-away is now known as al-maisan
=== wedgwood is now known as wedgwood_away
=== wedgwood_away is now known as wedgwood
=== deryck is now known as deryck[lunch]
=== yofel_ is now known as yofel
=== al-maisan is now known as almaisan-away
=== deryck[lunch] is now known as deryck
=== wedgwood is now known as wedgwood_away
=== almaisan-away is now known as al-maisan
=== wedgwood_away is now known as wedgwood
=== al-maisan is now known as almaisan-away

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!