| 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: | Whole Thread | Raw Message | 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
| 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? |