+++ b/configure
@@ -801,6 +805,7 @@ infodir
docdir
oldincludedir
includedir
+runstatedir
There's superfluous changes to ./configure unrelated to the changes in
configure.ac. Probably because you're using a different version of autotools,
or a vendor's patched copy. You can remove the changes with git checkout -p or
similar.
+++ b/src/backend/replication/basebackup_zstd.c
+bbsink *
+bbsink_zstd_new(bbsink *next, int compresslevel)
+{
+#ifndef HAVE_LIBZSTD
+ ereport(ERROR,
+ (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
+ errmsg("zstd compression is not supported by this
build")));
+#else
This should have an return; like what's added by 71cbbbbe8 and 302612a6c.
Also, the parens() around errcode aren't needed since last year.
+ bbsink_zstd *sink;
+
+ Assert(next != NULL);
+ Assert(compresslevel >= 0 && compresslevel <= 22);
+
+ if (compresslevel < 0 || compresslevel > 22)
+ ereport(ERROR,
This looks like dead code in assert builds.
If it's unreachable, it can be elog().
+ * Compress the input data to the output buffer until we run out of input
+ * data. Each time the output buffer falls below the compression bound for
+ * the input buffer, invoke the archive_contents() method for then next sink.
*the next sink ?
Does anyone plan to include this for pg15 ? If so, I think at least the WAL
compression should have support added too. I'd plan to rebase Michael's patch.
https://www.postgresql.org/message-id/[email protected]
--
Justin
