| From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
|---|---|
| To: | Daniil Zakhlystov <usernamedt(at)yandex-team(dot)ru> |
| Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org, Konstantin Knizhnik <knizhnik(at)garret(dot)ru> |
| Subject: | Re: libpq compression |
| Date: | 2021-03-19 01:02:32 |
| Message-ID: | 20210319010232.GH11765@telsasoft.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, Mar 18, 2021 at 07:30:09PM +0000, Daniil Zakhlystov wrote:
> The new status of this patch is: Ready for Committer
The CF bot is failing , because the last patch sent to the list is from January:
| Latest attachment (libpq-compression-31.patch) at 2021-01-12 14:05:22 from Konstantin Knizhnik ...
The most recent messages have all had links to github repos without patches
attached.
Also, the branches I've looked at on the github repos all have messy history,
and need to be squished into a coherent commit or set of commits.
Would you send a patch to the list with the commits as you propose to merge
them ?
Also, I'm not sure, but I think you may find that the zstd configure.ac should
use pkgconfig. This allowed the CIs to compile these patches. Without
pkg-config, the macos CI couldn't find (at least) LZ4.k
https://commitfest.postgresql.org/32/2813/
https://commitfest.postgresql.org/32/3015/
Also, in those patches, we have separate "not for merge" patches which enable
the compression library by default. This allows the CIs to exercise the
feature.
--
Justin
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Thomas Munro | 2021-03-19 01:16:19 | Re: fdatasync performance problem with large number of DB files |
| Previous Message | kuroda.hayato@fujitsu.com | 2021-03-19 01:02:20 | RE: [PATCH] pgbench: improve \sleep meta command |