Dear Bharath,
> > You mentioned at line 118, but at that time logical replication system is not
> created.
> > The subscriber is created at line 163.
> > Therefore WALs would not be consumed automatically.
>
> So, not calling pg_logical_slot_get_changes() on test_slot1 won't
> consume the WAL?
Yes. This slot was created manually and no one activated it automatically.
pg_logical_slot_get_changes() can consume WALs but never called.
>
> 2.
> +++ b/src/bin/pg_upgrade/t/003_logical_replication_slots.pl
>
> How about a more descriptive and pointed name for the TAP test file,
> something like 003_upgrade_logical_replication_slots.pl?
Good suggestion. Renamed.
> 3. Does this patch support upgrading of logical replication slots on a
> streaming standby? If yes, isn't it a good idea to add one test for
> upgrading standby with logical replication slots?
IIUC pg_upgrade would not be used for physical standby. The standby would be upgrade by:
* Recreating the database cluster, or
* Executing rsync command.
For more detail, please see the documentation.
Best Regards,
Hayato Kuroda
FUJITSU LIMITED