[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Maposmatic-dev] [PATCH maposmatic] Add a rendered maps feed
From: |
Maxime Petazzoni |
Subject: |
Re: [Maposmatic-dev] [PATCH maposmatic] Add a rendered maps feed |
Date: |
Wed, 10 Feb 2010 10:12:41 +0100 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
Hi,
* Thomas Petazzoni <address@hidden> [2010-02-09 22:36:31]:
> Ok for me, except one question :
>
> > + return (models.MapRenderingJob.objects
> > + .filter(status=2)
> > + .filter(resultmsg='ok')
> > + .order_by('-endofrendering_time')[:10])
>
> If I understand correctly, it means that at any given time, the RSS
> feed will only contains the 10 latest renderings. It means that if the
> mean rendering for the last 10 renderings is 4 minutes, they will
> disappear after 40 minutes.
>
> If the user reloads its RSS feed only every hour, or worse everyday,
> the user will miss a lot of renderings.
>
> So I think we should list a larger number of renderings, or list all
> renderings for the past 24/48 hours ?
That's a good point. But we need a combination of the two: all
renderings from the past 24 hours, but /at least/ ten of them, when
possible, otherwise we could end up with an empty feed (not likely, but
we never know).
Let's put this in first, I'll send a patch to fix that specific object
selection for the RSS feed later.
- Maxime
--
Maxime Petazzoni <http://www.bulix.org>
``One by one, the penguins took away my sanity.''
Linux kernel and software developer at MontaVista Software
signature.asc
Description: Digital signature