Great, thanks! On Feb 28, 2018 8:15 PM, "Joao De Almeida Pereira" < jdealmeidapere...@pivotal.io> wrote:
> Hello Harshal, > > I tried the example you showed and it works. Also passed the patch through > our CI Pipeline and everything is good. > > Thanks > Joao > > On Wed, Feb 28, 2018 at 6:25 AM Harshal Dhumal < > harshal.dhu...@enterprisedb.com> wrote: > >> Hi, >> >> On Tue, Feb 27, 2018 at 8:54 PM, Dave Page <dp...@pgadmin.org> wrote: >> >>> Hi >>> >>> On Tue, Feb 27, 2018 at 2:36 PM, Harshal Dhumal < >>> harshal.dhu...@enterprisedb.com> wrote: >>> >>>> Hi, >>>> >>>> Please find patch to fix wrong sql issue for time related type. >>>> >>>> Steps to reproduce: >>>> >>>> Alter any time/datetime array related data type, it generates sql with >>>> addition array bracket >>>> >>>> [image: Inline image 1] >>>> >>> >>> This seems to be missing the test case that you noted should be included >>> in the original bug report! >>> >> >> Please find updated patch with test cases. >> >> >> Thanks, >> >>> >>> -- >>> Dave Page >>> Blog: http://pgsnake.blogspot.com >>> Twitter: @pgsnake >>> >>> EnterpriseDB UK: http://www.enterprisedb.com >>> The Enterprise PostgreSQL Company >>> >>
