Re: Buffer usage in EXPLAIN and pg_stat_statements (review)

Euler Taveira de Oliveira <euler@timbira.com> wrote:

> > But there are some confusions in postgres; ReadBufferCount and
> > BufferHitCount are used for "get" and "hit", but "heap_blks_read"
> > and "heap_blks_hit" are used for "read" and "hit" in pg_statio_all_tables.
> I see. :(

I fixed the confusions of get, hit and read in your patch.
    long    num_hit = ReadBufferCount + ReadLocalBufferCount;
    long    num_read = num_hit - BufferHitCount - LocalBufferHitCount;
should be
    long    num_get = ReadBufferCount + ReadLocalBufferCount;
    long    num_hit = BufferHitCount + LocalBufferHitCount;
    long    num_read = num_get - num_hit;

ReadBufferCount means "number of buffer access" :(

Patch attached.

Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center