On Mon, 2007-05-28 at 21:25 +0100, Heikki Linnakangas wrote: > - RequestCheckpoint and CreateCheckpoint was changed to take a > CheckpointReason enum as argument, instead of boolean flags. This allows > printing a different message depending on the cause of checkpoint, and > is also cleaner IMHO.
Looks good. I'd be inclined to make the reason code for creating/dropping database the same and just say "Cluster DDL" or something, since for analysis purposes we just want to know it was DDL. I'd rather see the timings in seconds rather than ms. Counting significant digits will be a pain... -- Simon Riggs EnterpriseDB http://www.enterprisedb.com ---------------------------(end of broadcast)--------------------------- TIP 7: You can help support the PostgreSQL project by donating at http://www.postgresql.org/about/donate
