Re: NDirectFileRead and Write - Mailing list pgsql-hackers

From Tom Lane
Subject Re: NDirectFileRead and Write
Date
Msg-id 10637.1221657693@sss.pgh.pa.us
Whole thread Raw
In response to Re: NDirectFileRead and Write  (ITAGAKI Takahiro <itagaki.takahiro@oss.ntt.co.jp>)
List pgsql-hackers
ITAGAKI Takahiro <itagaki.takahiro@oss.ntt.co.jp> writes:
> - NDirectFile{Read|Write} are renamed to BufFile{Read|Write}Count.
> - They are still declared in execdebug.h and buffile.c includes it.

After some thought I concluded that the least ugly way to do this
was to declare and define the variables in the same places as the
other counters that are printed by ShowBufferUsage.  It's not any
worse from a modularity standpoint than the other alternatives we
considered, and it seems to satisfy the principle of least
surprise a little better.  Committed it that way.

> I did not touch messages in ShowBufferUsage() in the patch.

I didn't change the message either.  Heikki's suggestion of "temp
blocks" doesn't seem very good to me because of the likelihood
of confusion with temp tables.  I don't have a better alternative
offhand.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Andrew Chernow
Date:
Subject: Re: [PATCHES] libpq events patch (with sgml docs)
Next
From: Tom Lane
Date:
Subject: Re: [PATCHES] libpq events patch (with sgml docs)