pgsql: Code review for pgstat_get_crashed_backend_activity patch.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Code review for pgstat_get_crashed_backend_activity patch.
Date: 2011-10-21 20:36:16
Message-ID: E1RHLpA-0007f8-Aw@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Code review for pgstat_get_crashed_backend_activity patch.

Avoid possibly dumping core when pgstat_track_activity_query_size has a
less-than-default value; avoid uselessly searching for the query string
of a successfully-exited backend; don't bother putting out an ERRDETAIL if
we don't have a query to show; some other minor stylistic improvements.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/f9c92a5a3ead738c7de0dffa203a92b4d2fec413

Modified Files
--------------
src/backend/postmaster/pgstat.c | 54 ++++++++++++++++++++--------------
src/backend/postmaster/postmaster.c | 25 +++++++++------
src/backend/utils/adt/ascii.c | 29 ++++++++++--------
src/include/pgstat.h | 2 +-
src/include/utils/ascii.h | 3 +-
5 files changed, 66 insertions(+), 47 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Simon Riggs 2011-10-22 12:27:06 Re: pgsql: More cleanup after failed reduced-lock-levels-for-DDL feature.
Previous Message Tom Lane 2011-10-21 17:50:44 pgsql: More cleanup after failed reduced-lock-levels-for-DDL feature.