On Tue, Dec 22, 2009 at 3:27 AM, Takahiro Itagaki <itagaki.takah...@oss.ntt.co.jp> wrote: > Cedric Villemain <cedric.villem...@dalibo.com> wrote: >> Le vendredi 18 decembre 2009 09:44:40, Takahiro Itagaki a ecrit : >> > I'd like to add per-query buffer usage into contrib/pg_stat_statements. > > Here is a patch to add buffer usage columns into pg_stat_statements. > It also changes initialzation of the result TupleDesc from manually > coded routines to get_call_result_type().
I have reviewed this patch and I think it looks pretty good. A couple of minor nits: - There are needless whitespace changes in the definition of struct Counters. The changes to the existing four members should be reverted, and the new members should be made to match the existing members. - In the part that reads /* calc differences of buffer counters */, all the lines go past 80 columns. I wonder if it would be better to insert a line break just after the equals sign and indent the next line by an extra tab stop. See, e.g. src/backend/commands/user.c line 338. Other than that I think this is ready to commit. ...Robert -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
