Code review comment for lp://staging/~gary/launchpad/bug724025
Revision history for this message
| Jeroen T. Vermeulen (jtv) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
2eb648a
demo site
(Get the code!)
> If your question was rhetorical because you thought I *was* including makePackageBranch in the query count, it does
> not. The QueryCollector gets the queries from the most recent request, so the browser call I made is what we see in
> the results.
Indeed, I thought it counted all queries. Evidently I was wrong. (Which is ironic because I think I was the one to introduce query counts in tests over 3 years ago, and have done some work on the query collector).