| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
| Cc: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Alvaro Herrera from 2ndQuadrant <alvherre(at)alvh(dot)no-ip(dot)org>, Craig Ringer <craig(at)2ndquadrant(dot)com>, Ashutosh Sharma <ashu(dot)coek88(at)gmail(dot)com>, Julien Rouhaud <rjuju123(at)gmail(dot)com>, Peter Geoghegan <pg(at)bowt(dot)ie>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Abhijit Menon-Sen <ams(at)2ndquadrant(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Pavan Deolasee <pavan(dot)deolasee(at)2ndquadrant(dot)com> |
| Subject: | Re: [HACKERS] [PATCH] pageinspect function to decode infomasks |
| Date: | 2019-09-11 05:45:51 |
| Message-ID: | 20190911054551.GI1953@paquier.xyz |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, Sep 10, 2019 at 08:29:43AM +0530, Amit Kapila wrote:
> Good thought, but I think even if we want to change the name of
> tuple_data_split, it might be better done separately.
Yes, that's not the problem of this patch. Not sure if it actually
makes sense either to change it.
The regression tests added are rather unreadable when it comes to
print a lot of infomask flags. Could you add at least some unnest()
calls to the queries using heap_infomask_flags()? It would make the
diff lookup much more straight-forward to understand.
It would be good to comment as well what 2816 and 1080 stand for. The
current code makes it hard to understand for which purpose this is
used in the tests.
+ If decode_combined is set, combination flags like
Missing a markup here.
I am switching the patch as "waiting on author". Could you address
the comments raised please?
--
Michael
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Pavel Stehule | 2019-09-11 06:09:40 | Re: doc: update PL/pgSQL sample loop function |
| Previous Message | Amit Kapila | 2019-09-11 05:44:59 | Re: doc: update PL/pgSQL sample loop function |