> On 22. 11. 2021, at 16:44, Peter Eisentraut <peter.eisentraut@enterprisedb.com> wrote:
>
> On 22.11.21 01:47, Tomas Vondra wrote:
>> So I think just decoding the sequence tuples is a better solution - for large transactions (consuming many values
fromthe sequence) it may be more expensive (i.e. send more records to replica). But I doubt that matters too much -
it'slikely negligible compared to other data for large transactions.
>
> I agree that the original approach is better. It was worth trying out this alternative, but it seems quite
complicated. I note that a lot of additional code had to be added around several areas of the code, whereas the
originalpatch really just touched the logical decoding code, as it should. This doesn't prevent anyone from attempting
tooptimize things somehow in the future, but for now let's move forward with the simple approach.
+1
--
Petr Jelinek