public inbox for [email protected]  

help / color / mirror / Atom feed

From: Dave Page <[email protected]>
To: Joao De Almeida Pereira <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: [pgadmin][patch] [GreenPlum] When user press Explain Plan and Explain analyze plan an error is displayed
Date: Thu, 8 Feb 2018 07:55:46 +0000
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAE+jja=L+CpYv+6q1FkyYmvkAGTOsqCM+p3-e5EdLwcifz-Cmw@mail.gmail.com>
References: <CAE+jjakOhMJBMqsUaQuL9w4an0V4Q4nvY51K3GqWzm8jbj9cog@mail.gmail.com>
	<CAE+jja=n640Up0uWO=gQVp=A0rMJ-Y+YFSTCBjhwv98QLyRe2g@mail.gmail.com>
	<CAE+jja=L+CpYv+6q1FkyYmvkAGTOsqCM+p3-e5EdLwcifz-Cmw@mail.gmail.com>

It’s in my queue, but I’ve been traveling. I hope to get to it later today or tomorrow.

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

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

> On 7 Feb 2018, at 22:32, Joao De Almeida Pereira <[email protected]> wrote:
> 
> Hello,
> Any Feedback on this Patch?
> We have other changes that we already developed that depend on this change.
> 
> Thanks
> 
>> On Tue, Feb 6, 2018 at 9:33 AM Joao De Almeida Pereira <[email protected]> wrote:
>> Hello,
>> Any feedback on this Patch?
>> 
>> Thanks
>> Joao
>> 
>>> On Fri, Feb 2, 2018 at 5:50 PM Joao De Almeida Pereira <[email protected]> wrote:
>>> Hi Hackers,
>>> This is quite a big patch in order to solve the problem with the Explain Plan.
>>> 
>>> We sent 2 patches that have the following:
>>> - update-javascript-packages.diff 
>>>     Add package:
>>>      is-docker to select a specific setting when running the Chrome tests in
>>>      Docker
>>> 
>>>     Upgrade the version of:
>>>     - babel-loader
>>>     - extract-text-webpack-plugin
>>>     - jasmine-core
>>>     - jasmine-enzyme
>>>     - moment
>>> - explain-plan-greenplum.diff
>>>   Extract SQLEditor.execute and SQLEditor._poll into their own files and add test around them
>>>   Extract SQLEditor backend functions that start executing query to their own files and add tests around it
>>>   Move the Explain SQL from the front-end and now pass the Explain plan parameters as a JSON object in the start query call.
>>>   Extract the compile_template_name into a function that can be used by the different places that try to select the version of the template and the server type
>>> 
>>> 
>>> Thanks
>>> Joao

view thread (19+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: [pgadmin][patch] [GreenPlum] When user press Explain Plan and Explain analyze plan an error is displayed
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox