=== wedgwood is now known as wedgwood_away [00:07] The bugtask subscriber vocab is built in place [01:03] wgrant: mawson is updated, my patch cowboyed and JS unshelved, but it only renders timeouts [01:04] Ah, /auditor finally works [01:05] zomg' [01:05] lifeless: Ah, the auditor project page on DF [01:05] oh [01:20] wgrant: Right, so the exact matches are great for DF [01:20] Very fast [01:20] StevenK: I'd certainly hope so! [01:20] Maybe we want to restrict the MP vocab [01:21] One thing I thought of is that you probably want to do a case-insensitive match [01:21] On Branch.name [01:21] wgrant: So searching for 'devel' on proposing a LP branch returns too many matches [01:21] But I'd honestly expect to usually paste in a URL [01:22] StevenK: You're conditioned to paste a URL because search is so useless today [01:22] It usually times out, and when it doesn't time out it doesn't return useful results unless you're very accurate [01:23] Maybe we want to search on unique_name too [01:23] But I discounted that [01:24] Prefix search on unique_name makes sense if there's a / in the term [01:24] Otherwise I don't think it's valuable [01:27] wgrant: Thoughts on using BranchRestrictedOnProduct for BMP's vocab? [01:28] StevenK: Remember that BMPs work for more than products [01:28] wgrant: Prefix search? Then searching for launchpad/devel won't work [01:30] StevenK: That's true. [01:30] But mmm [01:30] contains_string is probably too heavy handed for unique_name [01:31] Yes, but it's all we have without schema changes [01:34] And I don't think it's going to be toooo long for a trigram to be effective [01:34] But you should try. [01:36] You mean like http://pastebin.ubuntu.com/1641700/ :-) [01:37] StevenK: Right, that sort of thing [01:37] Ideally a unique_name match would be component-wise rather than character-wise, but we can't really do that atm [01:38] Right [01:38] Let me patch that onto DF [01:38] wgrant: Oh, shall we TGRM up name and unique_name ? [01:39] StevenK: Try with and without [01:39] Max length of unique_name today is 141 [01:39] Which isn't great, but not fatal [01:40] What's that one? [01:40] ~openerp-commiter/openobject-addons/trunk-configuration_wizard_improvement-atp-improve-general-apply-button-color-in-configuration-wizard-pja | 141 [01:40] That's digusting [01:40] *disgusting [01:41] Then [01:41] ~78luphr0rnk2nuqimstywepozxn9kl19tqh0tx66b5dki1xxsh5m-launchpad-a811i2i3ytqlsztthjth0svbccw8inm65tmkqp9sarr553jq53in/syncany/delimiters | 135 [01:41] ~78luphr0rnk2nuqimstywepozxn9kl19tqh0tx66b5dki1xxs-phn5hho65-a811i2i3ytqlsztthjth0svbccw8inm65tmkqp9sarr553jq5/sysbench/sysbench-pg | 131 [01:41] Then more openerp [01:41] I can't complain about the 135 or 131, those are effectively our fault [01:42] Yep [01:42] https://code.launchpad.net/~chipaca/ubuntuone-client/dobey-needs-a-nice-self-contained-dbus-signal-to-let-the-applet-know-when-content-queue-changes [01:45] Apparently, the searches are all ~ 2 seconds [01:46] ie. awful [01:46] get a plan [01:49] wgrant: http://pastebin.ubuntu.com/1641721/ [01:49] StevenK: Heh [02:02] StevenK: FYI I prefer to name cowboyed indices and tables temp_* so they can be easily found and removed [02:06] It's still 440 ms with the TRGM [02:36] wgrant: Would gist rather than gin help? [02:36] StevenK: No [02:36] GiST vs GIN is mostly a write vs read tradeoff [02:36] GIN is slower at writes, but faster at reads [02:36] So we want GIN [02:38] But then we're stuffed even with the index [02:38] Not if it's a relatively consistent 4s [02:38] It's not create [02:38] great [02:38] But it's better than 2s consistent. [02:38] And will be faster on prod [02:38] StevenK: Is this unique_name or name? [02:39] The former [02:39] Which is rare [02:39] though most of the length will be in name too, see how it performs [02:39] That index is just creating now [02:42] TRGM for name is 300ms [02:42] Reasonable. [02:43] I was expecting more on the order of 100 [02:43] Not 300 or 400 [02:43] Not with a long large GIN index [02:44] What's the \d magic to see the index size? [02:44] \di+ [02:45] Holy crap, they're 200MiB all up [02:47] wgrant: So, push the changes for this branch, and create a new branch for two shiny TRGMs? [02:48] I think so [03:14] * StevenK stabs the branch scanner and reaches for carob [03:33] wgrant: https://code.launchpad.net/~stevenk/launchpad/branch-trgm-indicies/+merge/148082 [03:33] wgrant: And new-branch-search is updated === almaisan-away is now known as al-maisan === dimitern is now known as dimitern_afk [08:59] good morning === 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