=== alexlist` is now known as alexlist
StevenKwgrant: http://pastebin.ubuntu.com/5815654/02:56
wgrantStevenK: Right, that's broken03:11
StevenKwgrant: Broken how?03:12
wgrantStevenK: I'm not seeing how that query would always return the latest03:12
StevenKThe ORDER BY ?03:13
wgrantBut it doesn't pick just the latest03:13
wgrantIt returns all of them03:13
wgrantExcept if there are more than 5000, in which case it just semi-randomly won't return some of them03:13
StevenKSure, and then we loop through them, so the rejector cached property would have been set twice, and overwritten by the second one03:14
wgrantAh, because it's ascending, so that's even worse03:14
wgrantNot only is it going to be slow, but it will also ignore any recent event once more than 5000 would be returned03:15
StevenKwgrant: So, we want a DISTINCT ON object, operation ?03:16
StevenKWell, a distinct option to /fetch/03:17
wgrantI don't particularly care about implementation03:17
wgrantI'm presenting the constraints03:17
StevenKwgrant: I was envisioning being able to answer "Tell me everything that happened to <object>"03:19
wgrantStevenK: That's certainly a useful operation to have.03:19
wgrantBut it's not what we want here03:20
StevenKwgrant: Given any change is going to require changes to auditor, auditorclient and lp.services.auditor.client, I want to make sure it is going to be the right call for what we want before we start.03:20
wgrantThis is why I always say you should try to get a minimum viable implementation of infrastructure like this before you land it, before you make 4 auditorclient releases and 8 DB patches03:21
wgrantin two days03:21
wgrantThere are also considerations that must be made in terms of transactionality.03:22
StevenKwgrant: DB patches? Since when does auditor require them?03:45
wgrantIt hopefully won't, but it's similar in style to other large branch series that could03:46
wgrantSo it's the same concept03:46
StevenKwgrant: In terms of DB patches, which require timing, deployment, and merging in stable, sure, I agree. auditorclient requires a pushed branch, setup.py sdist and commiting the tarball to lp-sourcedpes.03:51
wgrantAnd getting to auditorclient 0.0.99999 before we deploy it :P03:56
StevenKDjango's query rubbish makes me miss storm03:57
StevenKOh, wonderful. Looks like sqlite does not support SELECT DISTINCT ON04:13
StevenKwgrant: So if sqlite does not support SELECT DISTINCT ON, and GROUP BY in Django looks ... fun, since they want you to use their aggregation functions, what are my options?04:46
wgrantStevenK: Possibly DISTINCT with a subquery05:10
* StevenK stabs QuerySet.values() being useless05:13
StevenKwgrant: I'm failing to see how that buys me anything, but that could be due to Django's ORM eating my brain05:15
StevenKq.values('object', 'operation').distinct() works, but if I call .values('id') on that, I get both06:31
=== tasdomas_afk is now known as tasdomas
=== jamesh_ is now known as jamesh
mwhudsonStevenK: you can use custom aggregate functions in django07:57
mwhudsonStevenK: http://voices.canonical.com/michael.hudson/2012/09/02/using-postgres-array_agg-from-django/07:57
StevenKmwhudson: This is the problem, though -- what I really want is DISTINCT ON, but I can't work out the right query when I'm forbidden from using it.08:18
=== tasdomas is now known as tasdomas_afk
=== tasdomas_afk is now known as tasdomas
lifelessStevenK: LOL - bug 119645510:20
_mup_Bug #1196455: power manegmant discrepanty <Auditor:New> <https://launchpad.net/bugs/1196455>10:20
=== wedgwood_away is now known as wedgwood
=== tasdomas is now known as tasdomas_afk
=== Gwaihir_ is now known as Gwaihir
johntroni'm trying to create a PPA for nginx that includes the SPDY and pagespeed modules. i've already built and installed from source, and everything works, so now i'm just trying to figure out how to package it. there's arleady an nginx package. i'm not sure how to specify the location of the upstream tarbal or how to build the dependencies during the build process21:44
johntroncan anyone help me figure out how to get the nginx tarball included correctly and how to get the dependencies `./configure`d correctly?21:47
=== wedgwood is now known as wedgwood_away

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