Re: [pgAdmin][RM7351]: Templates not displayed

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Pradip Parkale <pradip(dot)parkale(at)enterprisedb(dot)com>
Cc: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin][RM7351]: Templates not displayed
Date: 2022-05-17 08:20:35
Message-ID: CA+OCxoz-LsVYMpXGisZKyR_WPbqRx+a8Ba07C_7_EvDZ-HwfNw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Tue, 17 May 2022 at 08:08, Pradip Parkale <
pradip(dot)parkale(at)enterprisedb(dot)com> wrote:

> Hi Dave/Team,
>
> On Wed, May 11, 2022 at 4:37 PM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi,
>>
>> On Wed, May 11, 2022 at 4:03 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> On Wed, 11 May 2022 at 09:40, Aditya Toshniwal <
>>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Dave,
>>>>
>>>> On Wed, May 11, 2022 at 2:00 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>
>>>>>
>>>>>
>>>>> On Wed, 11 May 2022 at 09:24, Aditya Toshniwal <
>>>>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>>>>
>>>>>> Hi,
>>>>>>
>>>>>> On Wed, May 11, 2022 at 1:24 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>>>
>>>>>>> Hi
>>>>>>>
>>>>>>> I think this change needs some thought and discussion on how it
>>>>>>> should be implemented. pgAdmin has worked this way for over 20 years with
>>>>>>> only this one suggestion to change afaicr - and I for one don't suddenly
>>>>>>> want templates showing up amongst my other databases.
>>>>>>>
>>>>>> User defined template DBs cannot come under "Show system objects". I
>>>>>> mean, they're not system objects.
>>>>>>
>>>>>
>>>>> No they're not. But we have over 20 years of them being classed that
>>>>> way, and users may not suddenly want to see template1 (for example) listed
>>>>> amongst their databases. There are various things to think about here, for
>>>>> example:
>>>>>
>>>> No template1, template0 are system objects. They won't be visible. Only
>>>> new databases created manually and marked as template are excluded from
>>>> "Show system objects" .
>>>>
>>>
>>> template0 is. template1 is *by default*, but not if you drop and
>>> recreate it.
>>>
>> Didn't think in that way :)
>> In that case, we can add a new preference - "Show non-system/user defined
>> template databases?" explicitly. By default "yes", show them.
>> Along with a new icon.
>>
> Has this been finalized?
>

No - I haven't seen any proposals for what will be done, except for what
you've written below which no one has commented on yet.

> Could I add a new preference setting to show/hide user-defined template
> databases? The new icon for that may look like a 'T' on top of the current
> DB icon.
>

I think that's fine.

>
>
>>>
>>>>
>>>>> - Should there be a separate option to show/hide template databases?
>>>>> - If so, what should the default be (hint: I think yes, and off)
>>>>> - Should template databases have a different icon?
>>>>>
>>>> We can have this. Good to differentiate.
>>>>
>>>>> - Should they have their own collection node?
>>>>> - They can't be connected to, so much of what's on the database
>>>>> properties dialog won't work. Should they have their own properties
>>>>> dialogue design?
>>>>>
>>>> I can connect to template DBs in pgAdmin.
>>>>
>>>
>>> Yeah, disregard that. My brain was mixing up datistemplate and
>>> datallowconn.
>>>
>>>
>>>>
>>>>>
>>>>>
>>>>>>
>>>>>>> On Wed, 11 May 2022 at 07:12, Pradip Parkale <
>>>>>>> pradip(dot)parkale(at)enterprisedb(dot)com> wrote:
>>>>>>>
>>>>>>>> Hi Hackers,
>>>>>>>>
>>>>>>>> Please find the attached patch for # 7351:Templates not displayed.
>>>>>>>>
>>>>>>>> I have fixed below issues
>>>>>>>>
>>>>>>>> 1. Databases which are marked as template manually by the user
>>>>>>>> should be visible independent of - "Show System Objects".
>>>>>>>> 2. DB properties dialogs should allow you to change template
>>>>>>>> flag in edit mode.
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> --
>>>>>>>> Thanks & Regards,
>>>>>>>> Pradip Parkale
>>>>>>>> Software Engineer | EnterpriseDB Corporation
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Dave Page
>>>>>>> Blog: https://pgsnake.blogspot.com
>>>>>>> Twitter: @pgsnake
>>>>>>>
>>>>>>> EDB: https://www.enterprisedb.com
>>>>>>>
>>>>>>>
>>>>>>
>>>>>> --
>>>>>> Thanks,
>>>>>> Aditya Toshniwal
>>>>>> pgAdmin Hacker | Software Architect | *edbpostgres.com*
>>>>>> <http://edbpostgres.com>
>>>>>> "Don't Complain about Heat, Plant a TREE"
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Dave Page
>>>>> Blog: https://pgsnake.blogspot.com
>>>>> Twitter: @pgsnake
>>>>>
>>>>> EDB: https://www.enterprisedb.com
>>>>>
>>>>>
>>>>
>>>> --
>>>> Thanks,
>>>> Aditya Toshniwal
>>>> pgAdmin Hacker | Software Architect | *edbpostgres.com*
>>>> <http://edbpostgres.com>
>>>> "Don't Complain about Heat, Plant a TREE"
>>>>
>>>
>>>
>>> --
>>> Dave Page
>>> Blog: https://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EDB: https://www.enterprisedb.com
>>>
>>>
>>
>> --
>> Thanks,
>> Aditya Toshniwal
>> pgAdmin Hacker | Software Architect | *edbpostgres.com*
>> <http://edbpostgres.com>
>> "Don't Complain about Heat, Plant a TREE"
>>
>
>
> --
> Thanks & Regards,
> Pradip Parkale
> Software Engineer | EnterpriseDB Corporation
>

--
Dave Page
Blog: https://pgsnake.blogspot.com
Twitter: @pgsnake

EDB: https://www.enterprisedb.com

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Aditya Toshniwal 2022-05-17 11:55:59 [pgAdmin][RM7388] Error message will fill the entire window if the query is long
Previous Message Pradip Parkale 2022-05-17 07:08:38 Re: [pgAdmin][RM7351]: Templates not displayed