Re: head fails to build on SLES 12 (wal_compression=zstd)

2022-04-01 Thread Tom Lane
I wrote: > That seems like the appropriate answer to me. I verified that we > build cleanly and pass check-world against 1.4.0, and later I'm > going to set up BF member longfin to use that. Done ... > In short, I think we should push Justin's version-check patch, > and also fix the SGML docs to

Re: head fails to build on SLES 12 (wal_compression=zstd)

2022-03-31 Thread Michael Paquier
On Thu, Mar 31, 2022 at 09:10:00PM -0400, Tom Lane wrote: > In short, I think we should push Justin's version-check patch, > and also fix the SGML docs to say that we require zstd >= 1.4.0. 1.4.0 was released in April 2019, just 3 years ago. It does not sound that bad to me to make this version n

Re: head fails to build on SLES 12 (wal_compression=zstd)

2022-03-31 Thread Tom Lane
Robert Haas writes: > On Thu, Mar 31, 2022 at 7:38 PM Tom Lane wrote: >> Indeed. I tried building against 1.3.6 (mainly because it was laying >> around) and the error reported by Devrim is just the tip of the iceberg. >> ... >> I wonder whether Robert's ambition to be compatible with old version

Re: head fails to build on SLES 12 (wal_compression=zstd)

2022-03-31 Thread Robert Haas
On Thu, Mar 31, 2022 at 7:38 PM Tom Lane wrote: > Indeed. I tried building against 1.3.6 (mainly because it was laying > around) and the error reported by Devrim is just the tip of the iceberg. > With "make -k", I see unknown-symbol failures on > > ZSTD_CCtx_setParameter > ZSTD_c_compressionLevel

Re: head fails to build on SLES 12 (wal_compression=zstd)

2022-03-31 Thread Tom Lane
Justin Pryzby writes: > On Thu, Mar 31, 2022 at 11:44:40AM -0400, Tom Lane wrote: >> In view of 51c0d186d ("Allow parallel zstd compression"), I agree >> that some clarity about the minimum supported version of zstd >> seems essential. I don't want to be dealing with threading bugs >> in ancient

Re: head fails to build on SLES 12 (wal_compression=zstd)

2022-03-31 Thread Justin Pryzby
On Thu, Mar 31, 2022 at 11:44:40AM -0400, Tom Lane wrote: > Justin Pryzby writes: > > Possible responses look like: > > - Use 0 which also means "default" (need to verify that works across > > versions); > > - Or #ifndef ZSTD_CLEVEL_DEFAULT #define ZSTD_CLEVEL_DEFAULT 3; > > - Add a test for a

Re: head fails to build on SLES 12

2022-03-31 Thread Larry Rosenman
On 03/31/2022 2:58 pm, Andrew Dunstan wrote: On 3/31/22 10:23, Devrim Gündüz wrote: Hi, Latest snapshot tarball fails to build on SLES 12.5, which uses GCC 4.8-8. Build log is attached. Please let me know if you want me to provide more info. AFAICT we don't have any buildfarm animals curre

Re: head fails to build on SLES 12

2022-03-31 Thread Andrew Dunstan
On 3/31/22 10:23, Devrim Gündüz wrote: > Hi, > > Latest snapshot tarball fails to build on SLES 12.5, which uses GCC > 4.8-8. Build log is attached. > > Please let me know if you want me to provide more info. > AFAICT we don't have any buildfarm animals currently reporting on SLES 12 (and the o

Re: head fails to build on SLES 12 (wal_compression=zstd)

2022-03-31 Thread Robert Haas
On Thu, Mar 31, 2022 at 12:37 PM Tom Lane wrote: > Hm. After rereading 51c0d186d I see that we're not asking for > parallel compression unless the user tells us to, so I guess > our fallback answer for any complaints in that area can be > "if it hurts, don't do it". Right. We can also tell peopl

Re: head fails to build on SLES 12 (wal_compression=zstd)

2022-03-31 Thread Tom Lane
Robert Haas writes: > On Thu, Mar 31, 2022 at 11:44 AM Tom Lane wrote: >> In view of 51c0d186d ("Allow parallel zstd compression"), I agree >> that some clarity about the minimum supported version of zstd >> seems essential. I don't want to be dealing with threading bugs >> in ancient zstd versi

Re: head fails to build on SLES 12 (wal_compression=zstd)

2022-03-31 Thread Justin Pryzby
On Thu, Mar 31, 2022 at 12:26:46PM -0400, Robert Haas wrote: > On Thu, Mar 31, 2022 at 11:44 AM Tom Lane wrote: > > Justin Pryzby writes: > > > Possible responses look like: > > > - Use 0 which also means "default" (need to verify that works across > > > versions); > > > - Or #ifndef ZSTD_CLEV

Re: head fails to build on SLES 12 (wal_compression=zstd)

2022-03-31 Thread Robert Haas
On Thu, Mar 31, 2022 at 11:44 AM Tom Lane wrote: > Justin Pryzby writes: > > Possible responses look like: > > - Use 0 which also means "default" (need to verify that works across > > versions); > > - Or #ifndef ZSTD_CLEVEL_DEFAULT #define ZSTD_CLEVEL_DEFAULT 3; > > - Add a test for a minimum

Re: head fails to build on SLES 12 (wal_compression=zstd)

2022-03-31 Thread Tom Lane
Justin Pryzby writes: > Possible responses look like: > - Use 0 which also means "default" (need to verify that works across > versions); > - Or #ifndef ZSTD_CLEVEL_DEFAULT #define ZSTD_CLEVEL_DEFAULT 3; > - Add a test for a minimum zstd version v1.3.7. This may be a good idea for >v15 in

Re: head fails to build on SLES 12 (wal_compression=zstd)

2022-03-31 Thread Justin Pryzby
On Thu, Mar 31, 2022 at 03:38:39PM +0100, Devrim Gündüz wrote: > On Thu, 2022-03-31 at 10:26 -0400, Tom Lane wrote: > > Devrim =?ISO-8859-1?Q?G=FCnd=FCz?= writes: > > > Latest snapshot tarball fails to build on SLES 12.5, which uses GCC > > > 4.8-8. Build log is attached. > > > > Hmm, what versio

Re: head fails to build on SLES 12

2022-03-31 Thread Devrim Gündüz
Hi, On Thu, 2022-03-31 at 10:26 -0400, Tom Lane wrote: > Devrim =?ISO-8859-1?Q?G=FCnd=FCz?= writes: > > Latest snapshot tarball fails to build on SLES 12.5, which uses GCC > > 4.8-8. Build log is attached. > > Hmm, what version of libzstd is present? 1.3.3 Regards, -- Devrim Gündüz Open Sourc

Re: head fails to build on SLES 12

2022-03-31 Thread Tom Lane
Devrim =?ISO-8859-1?Q?G=FCnd=FCz?= writes: > Latest snapshot tarball fails to build on SLES 12.5, which uses GCC > 4.8-8. Build log is attached. Hmm, what version of libzstd is present? regards, tom lane

head fails to build on SLES 12

2022-03-31 Thread Devrim Gündüz
Hi, Latest snapshot tarball fails to build on SLES 12.5, which uses GCC 4.8-8. Build log is attached. Please let me know if you want me to provide more info. Thanks! Regards, -- Devrim Gündüz Open Source Solution Architect, PostgreSQL guy Twitter: @DevrimGunduz , @DevrimGunduzTR /usr/bin/clan