jangjang0401 commented on PR #1743:
URL: https://github.com/apache/cloudberry/pull/1743#issuecomment-4483392460

   > LGTM. Another approach could be to check `recovery.conf` for each database 
instance. However, we cannot do this using SQL. We must connect to each host 
and run a shell command. After that, we must check if the database is up and 
the replication slot is active. We should postpone checking `recovery. conf` 
until the current approach is no longer sufficient.
   
   Thank you for the review and for sharing the alternative approach.
   I agree with your suggestion to postpone the standby.signal check
   until the current approach is no longer sufficient. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to