From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [RM#4546] Add RE-SQL tests for Column node |
Date: | 2019-08-02 06:24:31 |
Message-ID: | CANxoLDdkqLS_aBGoLMEcJ5-8s+2kfn29_Cuk9vp2d=SNSWtZPw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Murtuza
Following test cases are missing:
- Primary Key column.
- Variable options.
- Precision and Collation.
- Identity column with 'GENERATED AS BY DEFAULT'.
- ALTER column with identity values like CYCLE, minimum, maximum etc..
instead of updating comments in such test cases.
- Generated expression should use existing columns if possible like
(col1 * col2).
Apart from that can you please look into *RM #4500 *which seems minor
issue. Please send separate patch for both.
On Thu, Aug 1, 2019 at 5:36 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi All
>
> I'll review this patch and give my review comments.
>
> On Wed, Jul 31, 2019 at 4:10 PM Murtuza Zabuawala <
> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>
>> Hi,
>>
>> PFA patch to add RE-SQL tests for Column node.
>>
>>
>> --
>> Regards,
>> Murtuza Zabuawala
>> EnterpriseDB: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>
--
*Thanks & Regards*
*Akshay Joshi*
*Sr. Software Architect*
*EnterpriseDB Software India Private Limited*
*Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Yosry Muhammad | 2019-08-02 06:51:05 | [Help] Modifying get_columns.sql template |
Previous Message | Aditya Toshniwal | 2019-08-02 05:25:00 | [pgAdmin][RM4552] Dragging the selected text in SQL editor throws console errors |