What happened to jsonb's JENTRY_ISFIRST?

Started by Tom Laneover 11 years ago3 messages
#1Tom Lane
tgl@sss.pgh.pa.us

jsonb.h claims that the high order bit of a JEntry word is set on the
first element of a JEntry array. However, AFAICS, JBE_ISFIRST() is
not used anywhere, which is a good thing because it refers to a constant
JENTRY_ISFIRST that's not defined anywhere. Is this comment just a leftover
from a convention that's been dropped, or is it still implemented but not
via this macro?

If the bit is unused now, should we be worrying about reclaiming it for
better use? Like say allowing jsonb's to be larger than just a quarter
of the maximum datum size?

regards, tom lane

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

#2Peter Geoghegan
pg@heroku.com
In reply to: Tom Lane (#1)
Re: What happened to jsonb's JENTRY_ISFIRST?

On Wed, Aug 13, 2014 at 3:47 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:

If the bit is unused now, should we be worrying about reclaiming it for
better use? Like say allowing jsonb's to be larger than just a quarter
of the maximum datum size?

Commit d9daff0e0cb15221789e6c50d9733c8754c054fb removed it. This is an
obsolete comment.

--
Peter Geoghegan

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

#3Heikki Linnakangas
hlinnakangas@vmware.com
In reply to: Peter Geoghegan (#2)
Re: What happened to jsonb's JENTRY_ISFIRST?

On 08/14/2014 02:45 AM, Peter Geoghegan wrote:

On Wed, Aug 13, 2014 at 3:47 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:

If the bit is unused now, should we be worrying about reclaiming it for
better use? Like say allowing jsonb's to be larger than just a quarter
of the maximum datum size?

Commit d9daff0e0cb15221789e6c50d9733c8754c054fb removed it. This is an
obsolete comment.

Yeah. I just noticed the same and replied in the other thread
(/messages/by-id/53EC8194.4020804@vmware.com). Note
to self: check all the mails in inbox before replying..

- Heikki

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers