On 4/21/2026 12:34 PM, Ayush Tiwari wrote:
> 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
Good catch. I agree the second one should be a detail. The first letter
has been capitalized, and a period is added. So, the patch looks good to me.
Regards,
Yuchen Li
