Re: Enums printout - possible bug

From: Harshal Dhumal <harshal(dot)dhumal(at)enterprisedb(dot)com>
To: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, Bartosz Dmytrak <bdmytrak(at)gmail(dot)com>, Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>, pgAdmin Support <pgadmin-support(at)postgresql(dot)org>
Subject: Re: Enums printout - possible bug
Date: 2017-12-11 11:54:34
Message-ID: CAFiP3vyWYfh1S2pZa2G_nMoh2pya_WgEiADJktmFiu_Q-e4mug@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers pgadmin-support

--
*Harshal Dhumal*
*Sr. Software Engineer*

EnterpriseDB India: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

On Sat, Dec 9, 2017 at 12:06 PM, Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:

>
>
> On Sat, Dec 9, 2017 at 11:50 AM, Harshal Dhumal <
> harshal(dot)dhumal(at)enterprisedb(dot)com> wrote:
>
>>
>> On Sat, Dec 9, 2017 at 11:30 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> Yeah, that one is in my list to work on. Have you specifically tested it
>>> with enum types (not enum[])?
>>>
>>
>> Yes.
>>
>
> @Harshal,
>
> Can you add multidimensional array, enum types into feature test
> (pg_datatype_validation_test.py) ?
>

I think I need to add test cases for all array data types not just enum
types since fix was not targeted for any particular data type.

>
> [image: Inline image 3]
>>
>>
>>
>>>
>>> On Sat, Dec 9, 2017 at 5:55 AM, Harshal Dhumal <
>>> harshal(dot)dhumal(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Dave,
>>>>
>>>> Recently I sent a patch
>>>> <https://www.postgresql.org/message-id/CAFiP3vzj6t2QuhkWy-sHnpcQB4tiq%2BK6gqLOVQkkcOuBtksvfw%40mail.gmail.com>
>>>> regarding multidimensional array representation issue.
>>>> In that patch I have reworked about how multidimensional (1 dimension
>>>> to n dimension)
>>>> array data should be represented in grid.
>>>> Also this patch covers almost all the array data types including
>>>> composite array data types like
>>>> int8range[], enum[], inet[], cidr[], macaddr[], uuid[], xml[], bit[],
>>>> varbit[] and so on.
>>>>
>>>> Please review the patch and let me know if any thing needs to be
>>>> included in this patch.
>>>>
>>>> Thanks,
>>>>
>>>>
>>>> --
>>>> *Harshal Dhumal*
>>>> *Sr. Software Engineer*
>>>>
>>>> EnterpriseDB India: http://www.enterprisedb.com
>>>> The Enterprise PostgreSQL Company
>>>>
>>>> On Sat, Dec 9, 2017 at 10:45 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>
>>>>> Murtuza, can you investigate please?
>>>>>
>>>>> Thanks!
>>>>>
>>>>> On Fri, Dec 8, 2017 at 8:08 PM, Bartosz Dmytrak <bdmytrak(at)gmail(dot)com>
>>>>> wrote:
>>>>>
>>>>>> Hi all,
>>>>>>
>>>>>> According to documentation: https://www.postgresql.org/doc
>>>>>> s/current/static/functions-enum.html when I execute SELECT
>>>>>> enum_range(null::rainbow) then output should look like this:
>>>>>> {red,orange,yellow,green,blue,purple}
>>>>>>
>>>>>> But in pgAdmin output looks like this: {{,r,e,d,,,o,r,a,n,g,e,,,y,e,l
>>>>>> ,l,o,w,,,g,r,e,e,n,,,b,l,u,e,,,p,u,r,p,l,e,}}. When you click on the
>>>>>> field, popup window shows correct value. I’ve double checked it in psql,
>>>>>> and output is correct. I think it could be considered as bug in pgAdmin
>>>>>>
>>>>>>
>>>>>>
>>>>>> PgAdmin details:
>>>>>>
>>>>>> *Version *2.0
>>>>>>
>>>>>> *Python Version *2.7.13 (v2.7.13:a06454b1afa1, Dec 17 2016,
>>>>>> 20:42:59) [MSC v.1500 32 bit (Intel)]
>>>>>>
>>>>>> *Flask Version *0.12.2
>>>>>>
>>>>>> *Application Mode *Desktop
>>>>>>
>>>>>>
>>>>>>
>>>>>> Pg Version: 10.1
>>>>>>
>>>>>> Regardless this one, many thanks for your great job pgAdmin Team!
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> Best regards,
>>>>>>
>>>>>> *Bartosz Dmytrak*
>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Dave Page
>>>>> Blog: http://pgsnake.blogspot.com
>>>>> Twitter: @pgsnake
>>>>>
>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>> The Enterprise PostgreSQL Company
>>>>>
>>>>
>>>>
>>>
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>
>>
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2017-12-11 12:01:59 pgAdmin 4 commit: Update query tool docs.
Previous Message Murtuza Zabuawala 2017-12-11 11:39:35 [pgAdmin4][Patch]: File manager - Fix logic to enable 'Select' button in List/Table view

Browse pgadmin-support by date

  From Date Subject
Next Message Dave Caughey 2017-12-11 14:20:02 Re: Wrong db/table listed in "View/Edit Data" | "Filter Rows..." tabs
Previous Message Murtuza Zabuawala 2017-12-11 05:29:13 Re: Wrong db/table listed in "View/Edit Data" | "Filter Rows..." tabs