On Sun, Mar 24, 2024 at 10:40 AM Amit Kapila <amit.kapil...@gmail.com> wrote: > > > For instance, setting last_inactive_time_1 to an invalid value fails > > with the following error: > > > > error running SQL: 'psql:<stdin>:1: ERROR: invalid input syntax for > > type timestamp with time zone: "foo" > > LINE 1: SELECT last_inactive_time > 'foo'::timestamptz FROM pg_repli... > > > > It would be found at a later point. It would be probably better to > verify immediately after the test that fetches the last_inactive_time > value.
Agree. I've added a few more checks explicitly to verify the last_inactive_time is sane with the following: qq[SELECT '$last_inactive_time'::timestamptz > to_timestamp(0) AND '$last_inactive_time'::timestamptz > '$slot_creation_time'::timestamptz;] I've attached the v18 patch set here. I've also addressed earlier review comments from Amit, Ajin Cherian. Note that I've added new invalidation mechanism tests in a separate TAP test file just because I don't want to clutter or bloat any of the existing files and spread tests for physical slots and logical slots into separate existing TAP files. -- Bharath Rupireddy PostgreSQL Contributors Team RDS Open Source Databases Amazon Web Services: https://aws.amazon.com
v18-0001-Track-last_inactive_time-in-pg_replication_slots.patch
Description: Binary data
v18-0002-Allow-setting-inactive_timeout-for-replication-s.patch
Description: Binary data
v18-0003-Introduce-new-SQL-funtion-pg_alter_replication_s.patch
Description: Binary data
v18-0004-Allow-setting-inactive_timeout-in-the-replicatio.patch
Description: Binary data
v18-0005-Add-inactive_timeout-based-replication-slot-inva.patch
Description: Binary data