pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Justin Pryzby <pryzby@telsasoft.com>
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Cc: Julien Rouhaud <rjuju123@gmail.com>
Cc: Dilip Kumar <dilipbalaut@gmail.com>
Cc: Kirill Bychik <kirill.bychik@gmail.com>
Cc: Fujii Masao <masao.fujii@oss.nttdata.com>
Subject: Re: User Interface for WAL usage data
Date: Fri, 3 Apr 2020 00:44:51 -0500
Message-ID: <20200403054451.GN14618@telsasoft.com> (raw)
In-Reply-To: <CAA4eK1+eP9rWx8PFHcGWmHiAicsBjFhpJc0KRdBLUOMs50iE0A@mail.gmail.com>
References: <CAA4eK1+o1Vj4Rso09pKOaKhY8QWTA0gWwCL3TGCi1rCLBBf-QQ@mail.gmail.com>
	<20200402054120.GC14618@telsasoft.com>
	<20200402.145826.78549992113208057.horikyota.ntt@gmail.com>
	<CAA4eK1J0CaomuePkTGG2OBEukeDmLFWQ4kP=+zvBcX324t=EUA@mail.gmail.com>
	<20200402063540.GE14618@telsasoft.com>
	<CAA4eK1JMvwBSvnPJTLipr4eKx99axxXzLjoH04q85NCFhndhyQ@mail.gmail.com>
	<20200403051111.GM14618@telsasoft.com>
	<CAA4eK1+eP9rWx8PFHcGWmHiAicsBjFhpJc0KRdBLUOMs50iE0A@mail.gmail.com>

On Fri, Apr 03, 2020 at 10:52:02AM +0530, Amit Kapila wrote:
> On Fri, Apr 3, 2020 at 10:41 AM Justin Pryzby <pryzby@telsasoft.com> wrote:
> >
> > > > > > > >    WAL: records=2359 full page records=42 bytes=447788
> >
> > 1) records; 2) pages ("full page images"); 3) bytes
> >
> > That is exactly like sort (method/type/size) and hash (buckets/batches/size),
> > and *not* like buffers, which shows various values all in units of "pages".
> >
> 
> The way you have written (2) appears to bit awkward. I would prefer
> "full page writes" or "full page images".

I didn't mean it to be the description used in the patch or anywhere else, just
the list of units.

I wonder if it should use colons instead of equals ?  As in:
|    WAL: Records: 2359  Full Page Images: 42  Size: 437kB

Note, that has: 1) two spaces; 2) capitalized "fields"; 3) size rather than
"bytes".  That's similar to Buckets:
|    Buckets: 1024  Batches: 1  Memory Usage: 44kB

I'm not sure if it should say "WAL: " or "WAL ", or perhaps "WAL:  "  If
there's no colon, then it looks like the first field is "WAL Records", but then
"size" isn't as tightly associated with WAL.  It could say:
|    WAL Records: n  Full Page Images: n  WAL Size: nkB

For comparison, buffers uses "equals" for the case showing multiple "fields",
which are all in units of pages:
|    Buffers: shared hit=15 read=2006

Also, for now, the output can be in kB, but I think in the future we should
take a recent suggestion from Andres to make an ExplainPropertyBytes() which
handles conversion to and display of a reasonable unit.  

-- 
Justin





view thread (16+ messages)  latest in thread

Message-ID: <20200403054451.GN14618@telsasoft.com>
Permalink:  ../20200403054451.GN14618@telsasoft.com/
Also on:    postgresql.org/message-id/20200403054451.GN14618@telsasoft.com

 · 

reply

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-hackers@postgresql.org
  Cc: pryzby@telsasoft.com, amit.kapila16@gmail.com, horikyota.ntt@gmail.com, pgsql-hackers@lists.postgresql.org, rjuju123@gmail.com, dilipbalaut@gmail.com, kirill.bychik@gmail.com, masao.fujii@oss.nttdata.com
  Subject: Re: User Interface for WAL usage data
  In-Reply-To: <20200403054451.GN14618@telsasoft.com>

* 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