postgres_fdw: Update comments in make_new_connection().

Expand the comment about the parallel_commit option to mention that the
default is false.

Also, since the comment about alteration of the keep_connections option,
which was located above the expanded comment, holds true for the
parallel_commit option, rewrite it to reflect this, and move it to after
the expanded comment.

Follow-up for commit 04e706d42.

Discussion: https://postgr.es/m/CAPmGK16Kg2Bf90sqzcZ4YM5cN_G-4h7wFUS01qQpqNB%2B2BG5_w%40mail.gmail.com

diff --git a/contrib/postgres_fdw/connection.c b/contrib/postgres_fdw/connection.c
index f9b8c01f3b95379abc17c2fe95c22ff9a992a2fa..541526ab80b2cbe62188749893293ac9b01739a5 100644 (file)
--- a/contrib/postgres_fdw/connection.c
+++ b/contrib/postgres_fdw/connection.c
@@ -318,14 +318,15 @@ make_new_connection(ConnCacheEntry *entry, UserMapping *user)
         * open even after the transaction using it ends, so that the subsequent
         * transactions can re-use it.
         *
-        * It's enough to determine this only when making new connection because
-        * all the connections to the foreign server whose keep_connections option
-        * is changed will be closed and re-made later.
-        *
         * By default, all the connections to any foreign servers are kept open.
         *
         * Also determine whether to commit (sub)transactions opened on the remote
-        * server in parallel at (sub)transaction end.
+        * server in parallel at (sub)transaction end, which is disabled by
+        * default.
+        *
+        * Note: it's enough to determine these only when making a new connection
+        * because these settings for it are changed, it will be closed and
+        * re-made later.
         */
        entry->keep_connections = true;
        entry->parallel_commit = false;