Re: Change the bool member of the Query structure to bits

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Quan Zongliang <quanzongliang(at)yeah(dot)net>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Change the bool member of the Query structure to bits
Date: 2024-02-20 15:45:51
Message-ID: 184953.1708443951@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Quan Zongliang <quanzongliang(at)yeah(dot)net> writes:
> The Query structure has an increasing number of bool attributes. This is
> likely to increase in the future. And they have the same properties.
> Wouldn't it be better to store them in bits? Common statements don't use
> them, so they have little impact. This also saves memory space.

I'm -1 on that, for three reasons:

* The amount of space saved is quite negligible. If queries had many
Query structs then it could matter, but they don't.

* This causes enough code churn to create a headache for back-patching.

* This'll completely destroy the readability of these flags in
pprint output.

I'm not greatly in love with the macro layer you propose, either,
but those details don't matter because I think we should just
leave well enough alone.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2024-02-20 15:53:58 Re: Missing Group Key in grouped aggregate
Previous Message Dean Rasheed 2024-02-20 15:29:51 Re: numeric_big in make check?