From: | Royce Ausburn <royce(dot)ml(at)inomial(dot)com> |
---|---|
To: | Greg Smith <greg(at)2ndquadrant(dot)com> |
Cc: | pgsql-rrreviewers(at)postgresql(dot)org |
Subject: | Re: Sign me up |
Date: | 2011-11-15 02:15:40 |
Message-ID: | 3C456296-7744-4C05-B36C-EFB809DF1884@inomial.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-rrreviewers |
On 15/11/2011, at 12:52 PM, Greg Smith wrote:
> On 11/14/2011 04:17 PM, Royce Ausburn wrote:
>> Please give me the simpler patches -- I've been using pgsql for years, but only read some pgsql code a couple of months ago ;) I'm not strong in C either, but I'll nut it out.
>>
>
> How about "Include detailed information about a row failing a CHECK constraint into the error message"? There's not a lot of code there, the main review work is preparing a test case that demonstrates the code does its job. (One really should have been included with the patch, but this condition isn't difficult to recreate) If that sounds reasonable, just edit the patch and add yourself there. I see you've already created your own entry there, so you know how that works.
Thanks Greg. I've added myself. I'll review it in the next couple of days.
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2011-11-15 22:04:58 | FlexLock patch |
Previous Message | Greg Smith | 2011-11-15 01:52:56 | Re: Sign me up |