I would not remove it altogether, there is plenty of consumers of this extension's output in the wild (even if I think it's unfortunate) that might not be interested in sequences, but changing it to off by default certainly makes sense.
-- Petr Jelinek > On 26. 1. 2022, at 11:09, Peter Eisentraut > <peter.eisentr...@enterprisedb.com> wrote: > > On 26.01.22 03:16, Tomas Vondra wrote: >> Here's a rebased version of the patch series. I decided to go back to the >> version from 2021/12/14, which does not include the changes to WAL logging. > > I notice that test_decoding still has skip-sequences enabled by default, "for > backward compatibility". I think we had concluded in previous discussions > that we don't need that. I would remove the option altogether. >
