From: Andres Freund <andres@anarazel.de>
To: Heikki Linnakangas <hlinnaka@iki.fi>
Cc: pgsql-bugs@postgresql.org
Cc: chjischj@163.com
Subject: Re: BUG #14687: pg_xlogdump does only count "main data" for record length and leading to incorrect statistics
Date: Mon, 5 Jun 2017 01:43:37 -0700
Message-ID: <20170605084337.fvm4nfa6c5hu3j2b@alap3.anarazel.de> (raw)
In-Reply-To: <cf787483-6848-c0a3-5e6e-71cc2cad3bb7@iki.fi>
References: <20170603165939.1436.58887@wrigleys.postgresql.org>
<20170604231012.rkhlpf5hqiiszwmb@alap3.anarazel.de>
<20170605022511.imow2t2ojsbauk72@alap3.anarazel.de>
<cf787483-6848-c0a3-5e6e-71cc2cad3bb7@iki.fi>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-bugs>
On 2017-06-05 11:23:43 +0300, Heikki Linnakangas wrote:
> On 06/05/2017 05:25 AM, Andres Freund wrote:
> > Hi Heikki,
> > Do you see any better way to get at the !fpi len than to iterate over
> > the blocks and sum up the image length? It's not really accurate to
> > display len (rec/tot) as total-fpi, total, but I don't have a better
> > backward-compatible idea? Actually, I don't even have a better idea for
> > master, so maybe it's ok as proposed?
>
> Your patch seems OK to me. Defining non-FPI size as (total_len - fpi_len)
> seems correct to me, I'm not sure what other definition you could have.
I'm not sure, that's why I chose the above. But you could argue that
the block ids etc. shouldn't be counted as part of either, or that the
previous displaying of record len / total len doesn't make much sense
anymore.
Greetings,
Andres Freund
--
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-bugs@postgresql.org
Cc: andres@anarazel.de, hlinnaka@iki.fi, chjischj@163.com
Subject: Re: BUG #14687: pg_xlogdump does only count "main data" for record length and leading to incorrect statistics
In-Reply-To: <20170605084337.fvm4nfa6c5hu3j2b@alap3.anarazel.de>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox