On Wed, Jul 10, 2019 at 12:22:02AM +0200, Laurenz Albe wrote: > Works for me. > > Marked as "ready for committer".
Hmm. synchronous_commit is user-settable, which means that it is possible to enforce a value in the connection string doing the connection. Isn't that something we had better enforce directly in the tool? In this case what could be fixed is GetConnection() which builds the connection string parameters. One thing that we would need to be careful about is that if the caller has provided a parameter for "options" (which is plausible as wal_sender_timeout is user-settable as of 12), then we need to make sure that the original value is preserved, and that the enforced of synchronous_commit is appended. Or, as you say, we just adjust the documentation. However I would recommend adding at least an example of connection string which uses "options" if the server sets synchronous_commit to "remote_apply" in this case. Still it seems to me that we have ways to reduce the confusion automatically. -- Michael
signature.asc
Description: PGP signature