Thread: pgsql: Publish checkpoint timing information to pg_stat_bgwriter.
Publish checkpoint timing information to pg_stat_bgwriter. Greg Smith, Peter Geoghegan, and Robert Haas Branch ------ master Details ------- http://git.postgresql.org/pg/commitdiff/b736aef2ec57b520ebadb70cceac0fed3168a5df Modified Files -------------- doc/src/sgml/monitoring.sgml | 22 ++++++++++++++++++++++ src/backend/access/transam/xlog.c | 30 +++++++++++++++++++++--------- src/backend/catalog/system_views.sql | 2 ++ src/backend/postmaster/pgstat.c | 2 ++ src/backend/storage/smgr/md.c | 25 ++++++++++--------------- src/backend/utils/adt/pgstatfuncs.c | 14 ++++++++++++++ src/include/catalog/catversion.h | 2 +- src/include/catalog/pg_proc.h | 4 ++++ src/include/pgstat.h | 4 ++++ src/test/regress/expected/rules.out | 2 +- 10 files changed, 81 insertions(+), 26 deletions(-)
On Thu, 2012-04-05 at 18:04 +0000, Robert Haas wrote: > Publish checkpoint timing information to pg_stat_bgwriter. > The pg_stat_bgwriter's definition is wrong. There is a column named checkopint_sync_time instead of checkpoint_sync_time. Small typo, I guess :) > Greg Smith, Peter Geoghegan, and Robert Haas > > Branch > ------ > master > > Details > ------- > http://git.postgresql.org/pg/commitdiff/b736aef2ec57b520ebadb70cceac0fed3168a5df > > Modified Files > -------------- > doc/src/sgml/monitoring.sgml | 22 ++++++++++++++++++++++ > src/backend/access/transam/xlog.c | 30 +++++++++++++++++++++--------- > src/backend/catalog/system_views.sql | 2 ++ > src/backend/postmaster/pgstat.c | 2 ++ > src/backend/storage/smgr/md.c | 25 ++++++++++--------------- > src/backend/utils/adt/pgstatfuncs.c | 14 ++++++++++++++ > src/include/catalog/catversion.h | 2 +- > src/include/catalog/pg_proc.h | 4 ++++ > src/include/pgstat.h | 4 ++++ > src/test/regress/expected/rules.out | 2 +- > 10 files changed, 81 insertions(+), 26 deletions(-) > > -- Guillaume http://blog.guillaume.lelarge.info http://www.dalibo.com
On Thu, Apr 5, 2012 at 4:52 PM, Guillaume Lelarge <guillaume@lelarge.info> wrote: > On Thu, 2012-04-05 at 18:04 +0000, Robert Haas wrote: >> Publish checkpoint timing information to pg_stat_bgwriter. > > The pg_stat_bgwriter's definition is wrong. There is a column named > checkopint_sync_time instead of checkpoint_sync_time. Small typo, I > guess :) Oops, how embarassing. Fixed (I hope). -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company