Re: Tomas Vondra > The one thing I don't quite like on the patch is that it actually > defaults to zstd before lz4. Which seems to not quite match the intent > to have both toast and wal default to the same thing. I don't have a > strong preference for either algorithm, but I also don't see why to put > zstd before lz4. I'd do it the other way around, i.e. lz4->zstd->pglz.
About the intent, I see it more as "PG19 is the version that switched away from pglz". It could be lz4 for both, or it could be lz4 for TOAST and zstd for WAL. There is some value in having both the same, but I'd defer to what the others are saying. I don't have a preference either way. This was just the last thing expressed here. (If we can't agree on that point, could we please still make the change and just toss a coin about this "bit"? :D) Christoph
