Remove some unnecessary tests of pgstat_track_counts.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 12 Oct 2010 18:44:25 +0000 (14:44 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 12 Oct 2010 18:44:25 +0000 (14:44 -0400)
commitf4d242ef94730c447d87b9840a40b0ec3371fe0f
treef6d6ea879c0ab7650966de4244218fe026589636
parent82659e0456f5c5a52a35ee41e63882c280ec2496
Remove some unnecessary tests of pgstat_track_counts.

We may as well make pgstat_count_heap_scan() and related macros just count
whenever rel->pgstat_info isn't null.  Testing pgstat_track_counts buys
nothing at all in the normal case where that flag is ON; and when it's OFF,
the pgstat_info link will be null, so it's still a useless test.

This change is unlikely to buy any noticeable performance improvement,
but a cycle shaved is a cycle earned; and my investigations earlier today
convinced me that we're down to the point where individual instructions in
the inner execution loops are starting to matter.
src/backend/postmaster/pgstat.c
src/include/pgstat.h