Re: [pgAdmin4][Patch] - RM 4104 & RM 3938 - Partition table fixes

From: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
To: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch] - RM 4104 & RM 3938 - Partition table fixes
Date: 2019-04-11 07:58:17
Message-ID: CANxoLDehZ38ENKvS8_NsbX2wHyTf_MFLtu0+vS3kB_PQ7tWH5A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Thanks patch applied.

On Thu, Apr 11, 2019 at 11:30 AM Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:

> Hi Akshay,
>
> On Thu, Apr 4, 2019 at 7:02 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
> wrote:
>
>> Hi Khushboo
>>
>> Below are the review comments:
>>
>> - Can you please add the help message for 'Default' in note control
>> as we do have for 'From/To' control. Also strings are not properly aligned
>> in the note control for Partition Keys and Partitions.
>>
>> Fixed.
>
>>
>> - Update the documentation and screenshot with the new changes.
>>
>> Done.
>
>>
>> - Found one issue, Steps to reproduce:
>> - Create a partition table with range partition and one child
>> partition of type 'Default'.
>> - Open the properties dialog of the newly created partition table
>> and add new child partition with 'Default' = No. Insert all the required
>> field 'Name', 'From' and 'To', still save button is disabled. I am not able
>> to create a new child partition.
>>
>> Fixed
>
>>
>> - If possible can you please fix the RM #*3327* which seems to be a
>> small fix, with the same patch.
>>
>> This fix will affect all the subnode controls as the code is generic. I
> will handle this separately.
>
> Thanks,
> Khushboo
>
>>
>> On Wed, Apr 3, 2019 at 3:42 PM Khushboo Vashi <
>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>
>>> Hi,
>>>
>>> Please find the attached patch to fix:
>>>
>>> 1. # 4104 - Edit data tables with primary key spanning multiple
>>> columns [Partition table]
>>> Issue: The partition table having multiple primary keys are not editable
>>> from the View data.
>>> 2. # 3938 - Properties of table with default partition not displayed
>>> Issue: The DEFAULT partition is not supported introduced in PG 11.
>>>
>>> Thanks,
>>> Khushboo
>>>
>>
>>
>> --
>> *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*

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2019-04-11 09:08:49 pgAdmin 4 commit: Depend on psycopg2-binary in the Python wheel, rather
Previous Message Akshay Joshi 2019-04-11 07:57:47 pgAdmin 4 commit: 1) Added support for Default Partition. Fixes #3938