Revert "pg_createsubscriber: Add log message when no publications exist to drop."

This reverts commit 74ac377d75135e02064fc4427bec401277b4f60c.

The previous change contained a misconception about how publications
are cleaned up on the subscriber. The newly added log message could
confuse users, particularly when running pg_createsubscriber with
--dry-run - users would see a "dropping publication" message
immediately followed by a "no publications found" message.

Discussion: https://postgr.es/m/CAHut+Pu7xz1LqNvyQyvSHrV0Sw6D=e6T-Jm=gh1MRJrkuWGyBQ@mail.gmail.com

diff --git a/src/bin/pg_basebackup/pg_createsubscriber.c b/src/bin/pg_basebackup/pg_createsubscriber.c
index b33566eabac268d3d563fe251f8a23f6b05448f4..d29407413d96b2207c78aa825ec1a38b3010d692 100644 (file)
--- a/src/bin/pg_basebackup/pg_createsubscriber.c
+++ b/src/bin/pg_basebackup/pg_createsubscriber.c
@@ -1755,15 +1755,10 @@ check_and_drop_publications(PGconn *conn, struct LogicalRepInfo *dbinfo)
            disconnect_database(conn, true);
        }
 
-       if (PQntuples(res) > 0)
-       {
-           /* Drop each publication */
-           for (int i = 0; i < PQntuples(res); i++)
-               drop_publication(conn, PQgetvalue(res, i, 0), dbinfo->dbname,
-                                &dbinfo->made_publication);
-       }
-       else
-           pg_log_info("no publications found");
+       /* Drop each publication */
+       for (int i = 0; i < PQntuples(res); i++)
+           drop_publication(conn, PQgetvalue(res, i, 0), dbinfo->dbname,
+                            &dbinfo->made_publication);
 
        PQclear(res);
    }