[PATCH] Fix duplicate errmsg in ALTER TABLE SPLIT PARTITION - Mailing list pgsql-hackers

From Ayush Tiwari
Subject [PATCH] Fix duplicate errmsg in ALTER TABLE SPLIT PARTITION
Date
Msg-id CAJTYsWUvMT5uKOasPnm6-o9CrdXbRONiAYHTKJb7wx66LB8S1A@mail.gmail.com
Whole thread
Responses Re: [PATCH] Fix duplicate errmsg in ALTER TABLE SPLIT PARTITION
Re: [PATCH] Fix duplicate errmsg in ALTER TABLE SPLIT PARTITION
List pgsql-hackers
Hi hackers,

The ereport() in transformPartitionCmdForSplit() that fires when
splitting a non-default partition while a default partition already
exists has two errmsg() calls:

    ereport(ERROR,
            errcode(ERRCODE_INVALID_OBJECT_DEFINITION),
            errmsg("can not split non-DEFAULT partition \"%s\"", ...),
            errmsg("new partition cannot be DEFAULT because ..."),
            parser_errposition(...));

The second one should had been errdetail()

The attached patch changes the second errmsg() to errdetail().

Also, I think "can not" can be replaced with "cannot" for consistency
throughout? Havent added is as part of this patch though. Thoughts?

Regards,
Ayush
Attachment

pgsql-hackers by date:

Previous
From: 반지현
Date:
Subject: Re: Return value of XLogInsertRecord() for XLOG_SWITCH record
Next
From: Ayush Tiwari
Date:
Subject: Re: [PATCH] postmaster: fix stale PM_STARTUP comment