On Mon, Jul 13, 2026 at 4:38 PM vignesh C <[email protected]> wrote:
>

Comments on 0002:
================
*
/*
- * Sequence synchronization relies on pg_get_sequence_data(), which
- * is only available since PostgreSQL 19.  Fail immediately with a
- * clear diagnosis instead of resetting the sequences to INIT state
- * and letting a sequencesync worker repeatedly fail trying to fetch
+ * Sequence synchronization relies on pg_get_sequence_data(), which is
+ * only available since PostgreSQL 19.  Fail immediately with a clear
+ * diagnosis instead of resetting the sequences to INIT state and
+ * letting a sequencesync worker repeatedly fail trying to fetch
  * sequence data with a query that can never succeed against this

This change doesn't seem to belong to this patch.

*
+ ereport(ERROR,
+ errcode(ERRCODE_OBJECT_NOT_IN_PREREQUISITE_STATE),
+ errmsg("cannot execute %s while a sequence synchronization worker
for subscription \"%s\" is still running",
+    "ALTER SUBSCRIPTION ... REFRESH SEQUENCES",

How about a slightly shorter message like: "cannot execute %s while a
sequence synchronization worker is running"?

-- 
With Regards,
Amit Kapila.


Reply via email to