On Fri, Jul 3, 2015 at 8:45 AM, Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Thu, Jul 2, 2015 at 7:44 PM, Alvaro Herrera <alvhe...@2ndquadrant.com> wrote: > > > > Can we reduce that to a single one? Maybe the > > first one could be errdetail or something. > > > > I think it is better other way (basically have second one as errdetail). > We already have one similar message in xlog.c that way. > ereport(LOG, > (errmsg("online backup mode canceled"), > errdetail("\"%s\" was renamed to \"%s\".", > BACKUP_LABEL_FILE, BACKUP_LABEL_OLD))); > > Attached patch consolidates errmsg into one message.
This can be tracked either in 9.5 Open Items or for next CF, any opinions? If nobody else has any opinion on this, I will add it to 9.5 Open Items list. With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com