Re: pageinspect patch, for showing tuple data - Mailing list pgsql-hackers

From Teodor Sigaev
Subject Re: pageinspect patch, for showing tuple data
Date
Msg-id 5655B243.9000407@sigaev.ru
Whole thread Raw
In response to Re: pageinspect patch, for showing tuple data  (Michael Paquier <michael.paquier@gmail.com>)
Responses Re: pageinspect patch, for showing tuple data  (Michael Paquier <michael.paquier@gmail.com>)
List pgsql-hackers
-                   bits_len = tuphdr->t_hoff -
-                       offsetof(HeapTupleHeaderData, t_bits);
+                   int bits_len =
+                           ((tuphdr->t_infomask2 & HEAP_NATTS_MASK) / 8 + 1) * 8;

As I understand offline comments of Nikolay, current version of page inspect 
contains an mistake here. Should we backpatch this?

> OK. I have switched the status of this patch to "Ready for committer"
> (please, committer-san, double-check the area around
> tuple_data_split_internal when fetching data for each attribute, I
> think that we got that right but I may be missing something as well).
Looks good for a first glance


-- 
Teodor Sigaev                                   E-mail: teodor@sigaev.ru
  WWW: http://www.sigaev.ru/
 



pgsql-hackers by date:

Previous
From: Grzegorz Sampolski
Date:
Subject: Re: pam auth - add rhost item
Next
From: Michael Paquier
Date:
Subject: Re: pageinspect patch, for showing tuple data