From: | Szymon Guz <mabewlun(at)gmail(dot)com> |
---|---|
To: | Robins Tharakan <tharakan(at)gmail(dot)com> |
Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Add more regression tests for CREATE OPERATOR |
Date: | 2013-06-26 07:29:38 |
Message-ID: | CAFjNrYto=0tQZaW_vj6-V1KodL2Z1CjkLAXwAnATQxebPAsAoA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
OK, so I think this patch can be committed, I will change the status.
thanks,
Szymon
On 26 June 2013 09:26, Robins Tharakan <tharakan(at)gmail(dot)com> wrote:
> Hi Szymon,
>
> The commented out test that you're referring to, is an existing test (not
> that I added or commented). I was going to remove but interestingly its
> testing a part of code where (prima-facie) it should fail, but it passes
> (probably why it was disabled in the first place)
> !
>
>
> So technically I hope this regression patch I submitted could go through
> since this feedback isn't towards that patch, but in my part I am quite
> intrigued about this test (and how it passes) and probably I'd get back on
> this thread about this particular commented out test in question, as time
> permits.
>
> --
> Robins Tharakan
>
>
> On 25 June 2013 04:12, Robins Tharakan <tharakan(at)gmail(dot)com> wrote:
>
>> Thanks a ton Szymon (for a reminder on this one).
>>
>> As a coincidental turn of events, I have had to travel half way across
>> the world and am without my personal laptop (without a linux distro etc.)
>> and just recovering from a jet-lag now.
>>
>> I'll try to install a VM on a make-shift laptop and get something going
>> to respond as soon as is possible.
>>
>> Thanks
>> --
>> Robins Tharakan
>>
>> --
>> Robins Tharakan
>>
>>
>> On 17 June 2013 05:19, Szymon Guz <mabewlun(at)gmail(dot)com> wrote:
>>
>>> On 23 May 2013 00:34, Robins Tharakan <tharakan(at)gmail(dot)com> wrote:
>>>
>>>> Hi,
>>>>
>>>> Please find attached a patch to take code-coverage of CREATE OPERATOR
>>>> (src/backend/commands/operatorcmds.c) from 56% to 91%.
>>>>
>>>> Any and all feedback is welcome.
>>>> --
>>>> Robins Tharakan
>>>>
>>>>
>>>> --
>>>> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
>>>> To make changes to your subscription:
>>>> http://www.postgresql.org/mailpref/pgsql-hackers
>>>>
>>>>
>>> Hi,
>>> there is one commented out test. I think it should be run, or deleted.
>>> There is no use of commented sql code which is not run.
>>>
>>> What do you think?
>>>
>>> regards,
>>> Szymon
>>>
>>
>>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2013-06-26 07:50:24 | Re: Review: Patch to compute Max LSN of Data Pages |
Previous Message | Robins Tharakan | 2013-06-26 07:26:33 | Re: Add more regression tests for CREATE OPERATOR |