On Thu, Sep 30, 2010 at 8:29 AM, Thom Brown <t...@linux.com> wrote: >>> This is a basic review of Fujii Masao's synchronous replication patch
Thanks for the review! And sorry for the delay. >>> I configured the primary's standby.conf to provide synchronous >>> replication to a single slave using fsync as it's replication level >>> and a timeout of 100ms. In the patch, standbys.conf still cannot accept a timeout setting. So please specify the standby name and replication level in that file. $PREFIX/share/standbys.conf.sample would be helpful. >>> The primary started up fine and was accepting connections. A base >>> backup was taken for the standby, and after configuring the standby, I >>> attempted to bring it up, but received the following error: >>> >>> postg...@cougar:~/project/data$ pg_ctl start >>> server starting >>> postg...@cougar:~/project/data$ LOG: database system was shut down in >>> recovery at 2010-09-29 22:52:24 BST >>> LOG: entering standby mode >>> LOG: redo starts at 0/1000020 >>> LOG: record with zero length at 0/10000B0 >>> FATAL: could not connect to the primary server: invalid connection >>> option "standby_name" >>> >>> I believe I am using the correct parameter as I followed the >>> additional documentation provided in the patch. >>> >>> >>> Conclusion >>> ========== >>> >>> It at least appears to me that it isn't functional in its current >>> state, or there is setup information missing from the >>> documentation.... or I've make a stupid mistake somewhere (likely). >> >> Quick back-peddle... it appears the patch was only successful on the >> standby. Only doc changes appeared to make it to the primary. >> Re-attempt tomorrow. Apologies. > > Well, that doesn't seem to have made any difference. Confirmed the > patch was applied in both cases, rebuilt, base backup again etc... no > change. Same error as in original review. *shrug* That's strange. I cannot reproduce that error. According to the error message, I guess that the patch has not been applied successfully to the master yet. Could you try applying the patch and re-installing the patch-applied-postgres again? Regards, -- Fujii Masao NIPPON TELEGRAPH AND TELEPHONE CORPORATION NTT Open Source Software Center -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers