postgres_fdw: Fix comments in connection.c.
authorEtsuro Fujita <efujita@postgresql.org>
Thu, 7 Oct 2021 09:15:00 +0000 (18:15 +0900)
committerEtsuro Fujita <efujita@postgresql.org>
Thu, 7 Oct 2021 09:15:00 +0000 (18:15 +0900)
Commit 27e1f1456 missed updating some comments.

Reviewed-by: Bharath Rupireddy
Backpatch-through: 14
Discussion: https://postgr.es/m/CAPmGK15Q2Nm6U%2Ba_GwskrWFEVBZ9_3VKOvRrprGufpx91M_3Sw%40mail.gmail.com

contrib/postgres_fdw/connection.c

index 1e67da983cb505cfb415ba5d9a348ba0446f33f1..ce0164e9d2364783256fb6a7a1cfe85324ff5cfb 100644 (file)
@@ -197,9 +197,9 @@ GetConnection(UserMapping *user, bool will_prep_stmt, PgFdwConnState **state)
                make_new_connection(entry, user);
 
        /*
-        * We check the health of the cached connection here when starting a new
-        * remote transaction. If a broken connection is detected, we try to
-        * reestablish a new connection later.
+        * We check the health of the cached connection here when using it.  In
+        * cases where we're out of all transactions, if a broken connection is
+        * detected, we try to reestablish a new connection later.
         */
        PG_TRY();
        {
@@ -215,9 +215,7 @@ GetConnection(UserMapping *user, bool will_prep_stmt, PgFdwConnState **state)
                ErrorData  *errdata = CopyErrorData();
 
                /*
-                * If connection failure is reported when starting a new remote
-                * transaction (not subtransaction), new connection will be
-                * reestablished later.
+                * Determine whether to try to reestablish the connection.
                 *
                 * After a broken connection is detected in libpq, any error other
                 * than connection failure (e.g., out-of-memory) can be thrown