Re: [RM#4546] Add RE-SQL tests for Column node
Akshay Joshi <[email protected]>
| Newsgroups | gmane.comp.db.postgresql.pgadmin.devel |
|---|---|
| Message-ID | <CANxoLDe7Ok5oP3D+9NkuKWcWQKzQ2qKdEtP6w8m7KF2=NvweYw@mail.gmail.com> |
Thanks patch applied. On Fri, Aug 16, 2019 at 7:31 AM Murtuza Zabuawala < [email protected]> wrote: > Hello Akshay, > > On Fri, Aug 2, 2019 at 11:54 AM Akshay Joshi < > [email protected]> wrote: > >> Hi Murtuza >> >> Following test cases are missing: >> >> - Primary Key column. >> >> There is no provision to set Primary key from Column dialog, This test > case will be covered with Primary key constraint RE-SQL tests. > >> >> - Variable options. >> >> Added > >> >> - Precision and Collation. >> >> Added > >> >> - Identity column with 'GENERATED AS BY DEFAULT'. >> >> Added > >> >> - ALTER column with identity values like CYCLE, minimum, maximum >> etc.. instead of updating comments in such test cases. >> >> Fixed > >> >> - Generated expression should use existing columns if possible like >> (col1 * col2). >> >> Fixed > > >> Apart from that can you please look into *RM #4500 *which seems minor >> issue. Please send separate patch for both. >> > > User is looking to provide length and precision for Integer type, which is > not possible, We can close this RM. > > > Please review the attached updated patch. > > > Regards, > Murtuza > > > >> >> On Thu, Aug 1, 2019 at 5:36 PM Akshay Joshi < >> [email protected]> wrote: >> >>> Hi All >>> >>> I'll review this patch and give my review comments. >>> >>> On Wed, Jul 31, 2019 at 4:10 PM Murtuza Zabuawala < >>> [email protected]> 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* >> > -- *Thanks & Regards* *Akshay Joshi* *Sr. Software Architect* *EnterpriseDB Software India Private Limited* *Mobile: +91 976-788-8246*