| From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
|---|---|
| To: | Michael Paquier <michael(at)paquier(dot)xyz> |
| Cc: | Justin Pryzby <pryzby(at)telsasoft(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: adding 'zstd' as a compression algorithm |
| Date: | 2022-02-16 15:40:01 |
| Message-ID: | CA+TgmobRisF-9ocqYDcMng6iSijGj1EZX99PgXA=3VVbWuahog@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, Feb 15, 2022 at 9:33 PM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> On Tue, Feb 15, 2022 at 02:33:32PM -0600, Justin Pryzby wrote:
> > What I mean is that any patches which *use* the zstd support should update
> > those for completeness.
> >
> > doc/src/sgml/config.sgml | 14 +++++++++-----
> > doc/src/sgml/install-windows.sgml | 2 +-
> > doc/src/sgml/installation.sgml | 5 +++--
>
> Yes, the patch misses the fact that each ./configure switch is
> documented. For consistency, I think that you should also add that in
> the MSVC scripts in the first version. It is really straight-forward
> to do so, and it should be just a matter of grepping for the code
> paths of lz4, then adjust things for zstd.
Makes sense. Here's an attempt by me to do that.
--
Robert Haas
EDB: http://www.enterprisedb.com
| Attachment | Content-Type | Size |
|---|---|---|
| v2-0001-Add-support-for-building-with-ZSTD.patch | application/octet-stream | 16.3 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Julien Rouhaud | 2022-02-16 15:42:46 | Re: pgsql: Move scanint8() to numutils.c |
| Previous Message | Alexander Pyhalov | 2022-02-16 15:08:30 | Re: postgres_fdw and skip locked |