On Tue, Mar 8, 2022 at 4:49 AM Jeevan Ladhe <jeevanladhe...@gmail.com> wrote: > I agree with your patch. The patch looks good to me. > Yes, the LZ4 flush check should also be fixed. Please find the attached > patch to fix the LZ4 code.
OK, committed all that stuff. I think we also need to fix one other thing. Right now, for LZ4 support we test HAVE_LIBLZ4, but TOAST and XLOG compression are testing USE_LZ4, so I think we should be doing the same here. And similarly I think we should be testing USE_ZSTD not HAVE_LIBZSTD. Patch for that attached. -- Robert Haas EDB: http://www.enterprisedb.com
fix-symbol-tests.patch
Description: Binary data
