Re: [pgadmin4][patch][GreenPlum] Display SQL for tables takes 15 minutes to execute

[Dave Page]

On Tue, Jun 12, 2018 at 6:34 AM, Ashesh Vashi  wrote:

> On Tue, Jun 12, 2018 at 10:04 AM, Ashesh Vashi <
> ashesh.va...@enterprisedb.com> wrote:
>
>> On Tue, Jun 12, 2018 at 10:02 AM, Robert Eckhardt 
>> wrote:
>>
>>>
>>>
>>> On Tue, Jun 12, 2018, 12:27 AM Ashesh Vashi <
>>> ashesh.va...@enterprisedb.com> wrote:
>>>
 Hi Joao,

 On Tue, Jun 12, 2018 at 1:50 AM, Joao De Almeida Pereira <
 jdealmeidapere...@pivotal.io> wrote:

> Hi Hackers,
>
> Attached you can find the diff file that corrects RM 3415
> 
>
 If I understand correctly, you're disabling the privileges for GPDB.
 Is that correct?

>>>
>>> It should be only for Greenplum.
>>>
>> Yeah - I got that.
>>
>>
>>> There was a performance optimization in Postgres 9.2 that make the
>>> current query in Postgres performant.
>>>
>> Ok.
>>
>> Changes looks good to me.
>>
> Committed!
>

Should this have included an updated to the release notes?

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

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

Re: [pgadmin4][patch][GreenPlum] Display SQL for tables takes 15 minutes to execute

[Ashesh Vashi]

On Tue, Jun 12, 2018 at 10:04 AM, Ashesh Vashi <
ashesh.va...@enterprisedb.com> wrote:

> On Tue, Jun 12, 2018 at 10:02 AM, Robert Eckhardt 
> wrote:
>
>>
>>
>> On Tue, Jun 12, 2018, 12:27 AM Ashesh Vashi <
>> ashesh.va...@enterprisedb.com> wrote:
>>
>>> Hi Joao,
>>>
>>> On Tue, Jun 12, 2018 at 1:50 AM, Joao De Almeida Pereira <
>>> jdealmeidapere...@pivotal.io> wrote:
>>>
 Hi Hackers,

 Attached you can find the diff file that corrects RM 3415
 

>>> If I understand correctly, you're disabling the privileges for GPDB.
>>> Is that correct?
>>>
>>
>> It should be only for Greenplum.
>>
> Yeah - I got that.
>
>
>> There was a performance optimization in Postgres 9.2 that make the
>> current query in Postgres performant.
>>
> Ok.
>
> Changes looks good to me.
>
Committed!

-- Thanks, Ashesh

>
> -- Thanks, Ashesh
>
>>
>> Rob
>>
>>
>>> -- Thanks, Ashesh
>>>
>>> ​
 Thanks
 Joao

>>>
>>>
>

Re: [pgadmin4][patch][GreenPlum] Display SQL for tables takes 15 minutes to execute

[Ashesh Vashi]

On Tue, Jun 12, 2018 at 10:02 AM, Robert Eckhardt 
wrote:

>
>
> On Tue, Jun 12, 2018, 12:27 AM Ashesh Vashi 
> wrote:
>
>> Hi Joao,
>>
>> On Tue, Jun 12, 2018 at 1:50 AM, Joao De Almeida Pereira <
>> jdealmeidapere...@pivotal.io> wrote:
>>
>>> Hi Hackers,
>>>
>>> Attached you can find the diff file that corrects RM 3415
>>> 
>>>
>> If I understand correctly, you're disabling the privileges for GPDB.
>> Is that correct?
>>
>
> It should be only for Greenplum.
>
Yeah - I got that.


> There was a performance optimization in Postgres 9.2 that make the current
> query in Postgres performant.
>
Ok.

Changes looks good to me.

-- Thanks, Ashesh

>
> Rob
>
>
>> -- Thanks, Ashesh
>>
>> ​
>>> Thanks
>>> Joao
>>>
>>
>>

Re: [pgadmin4][patch][GreenPlum] Display SQL for tables takes 15 minutes to execute

[Robert Eckhardt]

On Tue, Jun 12, 2018, 12:27 AM Ashesh Vashi 
wrote:

> Hi Joao,
>
> On Tue, Jun 12, 2018 at 1:50 AM, Joao De Almeida Pereira <
> jdealmeidapere...@pivotal.io> wrote:
>
>> Hi Hackers,
>>
>> Attached you can find the diff file that corrects RM 3415
>> 
>>
> If I understand correctly, you're disabling the privileges for GPDB.
> Is that correct?
>

It should be only for Greenplum. There was a performance optimization in
Postgres 9.2 that make the current query in Postgres performant.

Rob


> -- Thanks, Ashesh
>
> ​
>> Thanks
>> Joao
>>
>
>

Re: [pgadmin4][patch][GreenPlum] Display SQL for tables takes 15 minutes to execute

[Ashesh Vashi]

Hi Joao,

On Tue, Jun 12, 2018 at 1:50 AM, Joao De Almeida Pereira <
jdealmeidapere...@pivotal.io> wrote:

> Hi Hackers,
>
> Attached you can find the diff file that corrects RM 3415
> 
>
If I understand correctly, you're disabling the privileges for GPDB.
Is that correct?

-- Thanks, Ashesh

​
> Thanks
> Joao
>

[pgadmin4][patch][GreenPlum] Display SQL for tables takes 15 minutes to execute

[Joao De Almeida Pereira]

Hi Hackers,

Attached you can find the diff file that corrects RM 3415

​
Thanks
Joao


RM_3415.diff
Description: Binary data