Hi,
When we run ALTER SUBSCRIPTION ... REFRESH PUBLICATION and there is
an unkown table at local, it says;
NOTICE: added subscription for table public.tbl
I feel this a bit misleading because it says a subscription is added
but actually new subscription object is not created. Of cause, I can
understand the intention this message says, but I feel that
it is better to use other expression. For example, how about saying
NOTICE: table public.tbl is added to subscription sub1
as proposed in the attached patch. This also fixes the message
when a table is removed from a subscription.
Regards,
--
Yugo Nagata <[email protected]>
diff --git a/src/backend/commands/subscriptioncmds.c b/src/backend/commands/subscriptioncmds.c
index 6dc3f6e..5a7e6d5 100644
--- a/src/backend/commands/subscriptioncmds.c
+++ b/src/backend/commands/subscriptioncmds.c
@@ -573,9 +573,10 @@ AlterSubscription_refresh(Subscription *sub, bool copy_data)
copy_data ? SUBREL_STATE_INIT : SUBREL_STATE_READY,
InvalidXLogRecPtr, false);
ereport(NOTICE,
- (errmsg("added subscription for table %s.%s",
+ (errmsg("table %s.%s is added to subscription %s",
quote_identifier(rv->schemaname),
- quote_identifier(rv->relname))));
+ quote_identifier(rv->relname),
+ quote_identifier(sub->name))));
}
}
@@ -601,9 +602,10 @@ AlterSubscription_refresh(Subscription *sub, bool copy_data)
namespace = get_namespace_name(get_rel_namespace(relid));
ereport(NOTICE,
- (errmsg("removed subscription for table %s.%s",
+ (errmsg("table %s.%s is removed from subscription %s",
quote_identifier(namespace),
- quote_identifier(get_rel_name(relid)))));
+ quote_identifier(get_rel_name(relid)),
+ quote_identifier(sub->name))));
}
}
}
--
Sent via pgsql-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
