Code review comment for lp://staging/~gmb/launchpad/stored-proc-for-bug-heat-bug-582195

Revision history for this message
Björn Tillenius (bjornt) wrote :

On Wed, May 26, 2010 at 10:50:49AM -0000, Graham Binns wrote:
> The stored procedure is currently not hooked up to anything, but once
> the patch is applied it's easy to check the results of the stored
> procedure against the existing code using psql and an iharness session,
> thus:

Why not hook it up directly? It would be useful to have this tested,
since it's hard to check that it's doing what it should be doing.

--
Björn Tillenius | https://launchpad.net/~bjornt

« Back to merge proposal