Re: [PATCHES] Last infomask bit

2007-01-09 Thread Bruce Momjian
Patch applied. Thanks. I added a comment about the unused bits in the header file. --- Heikki Linnakangas wrote: Hi, We're running out of infomask bits in the tuple header. I bumped into this as I tried to apply

Re: [PATCHES] Last infomask bit

2007-01-09 Thread Tom Lane
Bruce Momjian [EMAIL PROTECTED] writes: Patch applied. Thanks. I added a comment about the unused bits in the header file. Has anyone bothered to measure the overhead added by having to mask to fetch or store the natts value? This is not a zero-cost improvement.

Re: [PATCHES] Last infomask bit

2007-01-05 Thread Tom Lane
Heikki Linnakangas [EMAIL PROTECTED] writes: I believe it would actually be even better to combine the t_natts and t_infomask fields to a single 32-bit infomask field. That's not happening, because the alignment is wrong ...unless maybe we switch this field to fall before t_ctid, but that