Comment 2 for bug 381360

The problem is this: the page contains a potentially large number of people links, all displayed with person/fmt:link. This formatter hits the ValidPersonCache table to calculate the proper icon, so even though the getSubscribers() method can be instrumented to retrieve the appropriate ValidPersonCache entry, because the interface is defined to just return the person, this information is thrown away, resulting in a re-query for each person displayed.

The proper fix is to batch the page, as we do elsewhere. Alternatively, we could change the API of getSubscribers() to return the formatted person link HTML and use a structured TAL in the page template. The latter is probably still not good since we could potentially have many thousands of people as members of the mailing list and such a big page is not very useful (and could still take a long time to load).