| From: | Aleksander Alekseev <aleksander(at)timescale(dot)com> |
|---|---|
| To: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Cc: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Zhihong Yu <zyu(at)yugabyte(dot)com>, Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com> |
| Subject: | Re: [PATCH] Compression dictionaries for JSONB |
| Date: | 2022-05-13 08:08:53 |
| Message-ID: | CAJ7c6TNBSN0BG_PJBz46mHMAtkam0MJKL5nWTBFhwKA+3N=4LA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi hackers,
> Here it the 2nd version of the patch:
>
> - Includes changes named above
> - Fixes a warning reported by cfbot
> - Fixes some FIXME's
> - The path includes some simple tests now
> - A proper commit message was added
Here is the rebased version of the patch. Changes compared to v2 are minimal.
> Open questions:
> - Dictionary entries are currently stored as NameData, the same type that is
> used for enums. Are we OK with the accompanying limitations? Any alternative
> suggestions?
> - All in all, am I moving the right direction?
I would like to receive a little bit more feedback before investing more time
into this effort. This will allow me, if necessary, to alter the overall design
more easily.
--
Best regards,
Aleksander Alekseev
| Attachment | Content-Type | Size |
|---|---|---|
| v3-0001-Compression-dictionaries-for-JSONB.patch | application/octet-stream | 50.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alvaro Herrera | 2022-05-13 08:30:12 | list of TransactionIds |
| Previous Message | Michael Paquier | 2022-05-13 07:09:00 | Re: Correct comment in ProcedureCreate() for pgstat_create_function() call. |