Re: unnecessary (same) restart_lsn handing in LogicalIncreaseRestartDecodingForSlot

2022-03-31 Thread Amit Kapila
On Wed, Mar 16, 2022 at 5:02 PM Ashutosh Bapat wrote: > > Hi All, > At the beginning of LogicalIncreaseRestartDecodingForSlot(), we have codeine > ``` > 1623 /* don't overwrite if have a newer restart lsn */ > 1624 if (restart_lsn <= slot->data.restart_lsn) > 1625 { > 1626 } > 1627

unnecessary (same) restart_lsn handing in LogicalIncreaseRestartDecodingForSlot

2022-03-16 Thread Ashutosh Bapat
Hi All, At the beginning of LogicalIncreaseRestartDecodingForSlot(), we have codeine ``` 1623 /* don't overwrite if have a newer restart lsn */ 1624 if (restart_lsn <= slot->data.restart_lsn) 1625 { 1626 } 1627 1628 /* 1629 * We might have already flushed far enough to dire