From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Emre Hasegeli <emre(at)hasegeli(dot)com> |
Cc: | Jim Nasby <Jim(dot)Nasby(at)bluetreble(dot)com>, Stephen Frost <sfrost(at)snowman(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Improve output of BitmapAnd EXPLAIN ANALYZE |
Date: | 2016-11-01 13:21:19 |
Message-ID: | CA+TgmoZqXBA_wJoDjVUvD_e1oGHhhX8_wWa2Nts=D4Uw0wphDA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Oct 31, 2016 at 6:56 AM, Emre Hasegeli <emre(at)hasegeli(dot)com> wrote:
> The BRIN Bitmap Index Scan has the same problem. I have seen people
> confused by this. I think N/A would clearly improve the situation.
I agree. Or perhaps better still, leave rows=%.0f out altogether when
we don't have a meaningful value to report. If it were OK to use some
unimportant-looking value as a proxy for "undefined", the SQL standard
wouldn't include nulls.
I don't like Tom's proposal of trying to fake up a value here when
EXPLAIN ANALYZE is in use. Reporting "exact" and "lossy" values for
BitmapAnd would be a fine enhancement, but artificially trying to
flatten that back into a row count is going to be confusing, not
helpful. (Just last week I saw a case where the fact that many pages
were being lossified caused a performance problem ... so treating
lossy pages as if they don't exist would have led to a lot of
head-scratching, because under Tom's proposal the row count would have
been way off.)
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2016-11-01 13:24:42 | Re: Patch: Implement failover on libpq connect level. |
Previous Message | Julian Markwort | 2016-11-01 13:20:43 | Re: [PATCH] pgpassfile connection option |