| === tasdomas_afk is now known as tasdomas | ||
| StevenK | allow-picked-versions = false now complains about setuptools 0.7.4 no matter what versions.cfg states. | 02:54 |
|---|---|---|
| * StevenK stabs buildout for being obtuse and obstreperous. | 02:54 | |
| StevenK | wgrant: I've been victorious over buildout | 03:09 |
| wgrant | Great | 03:09 |
| StevenK | wgrant: https://code.launchpad.net/~stevenk/launchpad/export-pu-auditor/+merge/171460 | 04:58 |
| wgrant | StevenK: Do the bulk auditor load calls also bulk load the relevant people? | 05:20 |
| wgrant | Are the auditor calls logged in the statement log? | 05:21 |
| wgrant | What are the timeouts on the auditor calls? | 05:21 |
| StevenK | wgrant: I don't think the enterpriseid stuff is that smart | 05:22 |
| wgrant | It'll need to be | 05:25 |
| StevenK | wgrant: Have you read enterpriseid_to_object() ? | 05:29 |
| wgrant | StevenK: Not in maybe a year. Why? | 05:32 |
| StevenK | wgrant: Because I can't think of a way given a list of enterpriseids to objectify them in one or two queries | 05:33 |
| wgrant | StevenK: That doesn't obviate the need to do it in one or two queries. | 05:34 |
| StevenK | Certainly not | 05:34 |
| wgrant | You'd want to split them up by class, then bulk load | 05:35 |
| wgrant | You'd also ideally introspect the storm cache to grab those that exist. | 05:35 |
| wgrant | But that might be a bit difficult | 05:35 |
| wgrant | StevenK: https://code.launchpad.net/~wgrant/launchpad/sha256-archiveuploader-pre/+merge/171464 https://code.launchpad.net/~wgrant/launchpad/sha256-archiveuploader/+merge/171465 | 06:24 |
| === tasdomas is now known as tasdomas_afk | ||
| StevenK | wgrant: r=me for the pre branch | 06:31 |
| StevenK | wgrant: And slamming you with Needs Fixing on the second | 06:39 |
| wgrant | That was more approve with comments material, but OK :) | 06:43 |
| === tasdomas_afk is now known as tasdomas | ||
| wgrant | StevenK: Updated | 07:04 |
| StevenK | wgrant: r=me | 07:09 |
| StevenK | wgrant: Are you going to bend gina to your will in terms of hashes, too? | 07:09 |
| wgrant | No comment | 07:10 |
| wgrant | But I might look at switching it to use some of the archiveuploader infrastructure | 07:11 |
| wgrant | Rather than reimplementing the world | 07:11 |
| StevenK | Sounds like a good idea | 07:11 |
| ttx | wgrant: ok, I'll try to drop the subscriber and see how it goes :) | 08:23 |
| ttx | wgrant: looks like it's passing tests now: https://code.launchpad.net/~ttx/launchpad/lp1193389/+merge/171491 | 09:23 |
| wgrant | ttx: Looking good, thanks. I'll review it properly tomorrow. | 11:17 |
| ttx | wgrant: awesome, thx | 11:40 |
| === tasdomas is now known as tasdomas_afk | ||
| === Ursinha is now known as Ursinha-afk | ||
| === Ursinha-afk is now known as Ursinha | ||
| === wedgwood_away is now known as wedgwood | ||
| === gary_poster is now known as gary_poster|away | ||
| === gary_poster|away is now known as gary_poster | ||
| === matsubara is now known as matsubara-lunch | ||
| === matsubara-lunch is now known as matsubara | ||
| === tasdomas_afk is now known as tasdomas | ||
| === tasdomas is now known as tasdomas_afk | ||
| === BradCrittenden is now known as bac | ||
| === wedgwood is now known as wedgwood_away | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!