Comment 8 for bug 483931

Revision history for this message
James Westby (james-w) wrote : Re: [Bug 483931] Re: No +activereviews page for Ubuntu

On Wed Dec 09 14:43:38 UTC 2009 Aaron Bentley wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> James Westby wrote:
> > On Wed Dec 09 05:30:33 UTC 2009 Jonathan Lange wrote:
> > That's fair enough though. We need to be able to access the info. Imagine
> > if you were told that you weren't allowed to get the full list of bugs filed
> > against LP, except by individually querying the bugs filed against it by
> > every LP user in turn. Silly, I know, but we need this information.
>
> I think you're assuming that +activereviews is only way to find this
> information in bulk. For other branch listings, we have a +merges page
> as well, which presents all the same code reviews as +activereviews (and
> more), organized differently and with batching. I think that would be
> much more suitable that +activereviews for this purpose.

Does this page exist for Ubuntu? Where is it linked from?

> > That would probably cause more load anyway, as we do the loops to extract
> > the full list for display elsewhere.
>
> In addition to load, +activereviews would cause database timeouts. Even
> if we provided +activereviews, it's unlikely it would ever work.
> Looping is bad, but it would actually work, so it would be better than
> +activereviews.
>
> >> >From a user PoV, we don't want to have a page of possible things to do
> >> that can practically grow without bound. We already have something like
> >> that on the bug tracker, and it's not terribly popular :P
> >
> > That we can work on. We have plans for LP external tools to help with things
> > like this. Some of the knowledge of that can feed back in to LP, and help
> > to design pages that show the user the most relevant things to them.
>
> We have a per-user page also:
> https://edge.launchpad.net/~james-w/+activereviews

Yes, and I think this is actually the better way to find reviews. It doesn't
currently suit my needs fully though, as there isn't a "Reviews I can do."
That then doesn't work too well for package branches, partly because
of the way in which upload permissions are structured in Ubuntu. It would be
fantastic to fix that, which is what I was referring to.

Thanks,

James