]> granicus.if.org Git - postgresql/commit
Redesign pageinspect function printing infomask bits
authorMichael Paquier <michael@paquier.xyz>
Thu, 19 Sep 2019 02:01:52 +0000 (11:01 +0900)
committerMichael Paquier <michael@paquier.xyz>
Thu, 19 Sep 2019 02:01:52 +0000 (11:01 +0900)
commit58b4cb30a5bf52d71a4d0e5f9f7e1da3e64f67cc
tree94866dcd6aa67583a626306fa96169b44de1e934
parent59354ccef5d7671bb11982628d6ddd6fffbad2c4
Redesign pageinspect function printing infomask bits

After more discussion, the new function added by ddbd5d8 could have been
designed in a better way.  Based on an idea from Álvaro, instead of
returning one column which includes both the raw and combined flags, use
two columns, with one for the raw flags and one for the combined flags.

This also takes care of some issues with HEAP_LOCKED_UPGRADED and
HEAP_XMAX_IS_LOCKED_ONLY which are not really combined flags as they
depend on conditions defined by other raw bits, as mentioned by Amit.

While on it, fix an extra issue with combined flags.  A combined flag
was returned if at least one of its bits was set, but all its bits need
to be set to include it in the result.

Author: Michael Paquier
Reviewed-by: Álvaro Herrera, Amit Kapila
Discussion: https://postgr.es/m/20190913114950.GA3824@alvherre.pgsql
contrib/pageinspect/expected/page.out
contrib/pageinspect/heapfuncs.c
contrib/pageinspect/pageinspect--1.7--1.8.sql
contrib/pageinspect/sql/page.sql
doc/src/sgml/pageinspect.sgml