Re: [pgAdmin][RM5569] [SQL] Bad display of tables storage parameters in the "SQL" tab (correction of bug #5398 not complete)

From: Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com>
To: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
Cc: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin][RM5569] [SQL] Bad display of tables storage parameters in the "SQL" tab (correction of bug #5398 not complete)
Date: 2020-07-10 11:48:33
Message-ID: CAKtn9dNUSS_JA1U-MzmmY9DFFqNgq18h1gTvpFti5qJGLhg6JA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Khushboo,

I had already corrected the patch as soon as I saw your mail, but couldn't
send you as I went out on an urgency. Now sending it.

On Fri, Jul 10, 2020 at 10:25 AM Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:

> Hi Rahul,
>
> I am getting errors while applying the patch, please rebase the patch and
> send it again.
>
> Thanks,
> Khushboo
>
> On Thu, Jul 9, 2020 at 1:25 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
> wrote:
>
>> Hi Khushboo
>>
>> Can you please review it.
>>
>> On Wed, Jul 8, 2020 at 6:12 PM Rahul Shirsat <
>> rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Hackers,
>>>
>>> Following patch includes fix for :
>>>
>>> Storage parameters should be displayed in parent tables's SQL.
>>>
>>> As the code will be upgraded in feature #5649
>>> <https://redmine.postgresql.org/issues/5649>, so only initial test
>>> cases for pg 10, 11 are included in this patch, so later it can be
>>> continued with the latest code.
>>>
>>> --
>>> *Rahul Shirsat*
>>> Software Engineer | EnterpriseDB Corporation.
>>>
>>> On Mon, Jul 6, 2020 at 2:27 PM Khushboo Vashi <
>>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Rahul,
>>>>
>>>> - The storage parameters should be displayed in the parent table's SQL
>>>> too
>>>> - Please write test cases, preferably re-sql test cases
>>>>
>>>> Thanks,
>>>> Khushboo
>>>>
>>>> On Mon, Jul 6, 2020 at 11:28 AM Akshay Joshi <
>>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hi Khushboo,
>>>>>
>>>>> Can you please review it.
>>>>>
>>>>> On Sat, Jul 4, 2020 at 1:45 AM Rahul Shirsat <
>>>>> rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:
>>>>>
>>>>>> Hi Hackers,
>>>>>>
>>>>>> Please find the attached patch which fixes the issue related to the
>>>>>> partition table with storage parameters.
>>>>>>
>>>>>> --
>>>>>> *Rahul Shirsat*
>>>>>> Software Engineer | EnterpriseDB Corporation.
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> *Thanks & Regards*
>>>>> *Akshay Joshi*
>>>>>
>>>>> *Sr. Software Architect*
>>>>> *EnterpriseDB Software India Private Limited*
>>>>> *Mobile: +91 976-788-8246*
>>>>>
>>>>
>>>
>>> --
>>> *Rahul Shirsat*
>>> Software Engineer | EnterpriseDB Corporation.
>>>
>>
>>
>> --
>> *Thanks & Regards*
>> *Akshay Joshi*
>>
>> *Sr. Software Architect*
>> *EnterpriseDB Software India Private Limited*
>> *Mobile: +91 976-788-8246*
>>
>

--
*Rahul Shirsat*
Software Engineer | EnterpriseDB Corporation.

Attachment Content-Type Size
RM5569_v3.patch application/octet-stream 55.0 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Khushboo Vashi 2020-07-10 12:01:58 Re: [pgAdmin][RM5569] [SQL] Bad display of tables storage parameters in the "SQL" tab (correction of bug #5398 not complete)
Previous Message Akshay Joshi 2020-07-10 06:21:40 Re: [pgAdmin4][Patch] - SonarQube Fixes