postgres_fdw: Fix comments in connection.c.
authorEtsuro Fujita <[email protected]>
Thu, 7 Oct 2021 09:15:00 +0000 (18:15 +0900)
committerEtsuro Fujita <[email protected]>
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