Dear Euler,

I love your proposal, so I want to join the review. Here are my first comments.

01. 
Should we restrict that `--subscriber-conninfo` must not have hostname or IP?
We want users to execute pg_subscriber on the target, right?

02.
When the application was executed, many outputs filled my screen. Some of them
were by pg_subscriber, and others were server log. Can we record them into
separated file? I imagined like pg_upgrade.

03.
A replication command is used when replication slots are created. Is there a
reason to use it? I think we do not have to use logical replication walsender 
mode,
we can use an SQL function instead. pg_create_logical_replication_slot() also 
outputs
LSN, isn't it sufficient?

04.
As you know, there are several options for 
publications/subscriptions/replication
slots. Do you have a good way to specify them in your mind? 

05.
I found that the connection string for each subscriptions have a setting
"fallback_application_name=pg_subscriber". Can we remove it?

```
postgres=# SELECT subconninfo FROM pg_subscription;
                                   subconninfo                                  
 
---------------------------------------------------------------------------------
 user=postgres port=5431 fallback_application_name=pg_subscriber dbname=postgres
(1 row)
```

Best Regards,
Hayato Kuroda
FUJITSU LIMITED



Reply via email to