Re: [HACKERS] error detail when partition not found

2017-03-02 Thread Amit Langote
On 2017/03/03 12:43, Robert Haas wrote: > On Mon, Feb 27, 2017 at 9:54 AM, Amit Langote > wrote: >> Updated patch is attached. > > Committed with one grammatical change to the comments. Thanks. I've marked this as fixed on the open item wiki page. Thanks, Amit

Re: [HACKERS] error detail when partition not found

2017-03-02 Thread Robert Haas
On Mon, Feb 27, 2017 at 9:54 AM, Amit Langote wrote: > Updated patch is attached. Committed with one grammatical change to the comments. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers

Re: [HACKERS] error detail when partition not found

2017-02-26 Thread Amit Langote
Thanks for the review. On 2017/02/27 2:39, Robert Haas wrote: > On Tue, Feb 21, 2017 at 7:28 AM, Amit Langote > wrote: >> Simon pointed out in a nearby thread [0] that the detail part of >> partition-not-found error should show just the partition keys. I posted a

Re: [HACKERS] error detail when partition not found

2017-02-26 Thread Robert Haas
On Tue, Feb 21, 2017 at 7:28 AM, Amit Langote wrote: > Simon pointed out in a nearby thread [0] that the detail part of > partition-not-found error should show just the partition keys. I posted a > patch on that thread [1], but to avoid confusion being caused by

[HACKERS] error detail when partition not found

2017-02-20 Thread Amit Langote
Simon pointed out in a nearby thread [0] that the detail part of partition-not-found error should show just the partition keys. I posted a patch on that thread [1], but to avoid confusion being caused by multitude of patches over there I'm re-posting it here. * What the patch does: Currently we