[pgadmin][patch] [GreenPlum] When user press Explain Plan and Explain analyze plan an error is displayed

From: Joao De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io>
To: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: [pgadmin][patch] [GreenPlum] When user press Explain Plan and Explain analyze plan an error is displayed
Date: 2018-02-02 22:50:30
Message-ID: CAE+jjakOhMJBMqsUaQuL9w4an0V4Q4nvY51K3GqWzm8jbj9cog@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

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

Attachment Content-Type Size
explain-plan-greenplum.diff application/octet-stream 182.4 KB
update-javascript-packages.diff application/octet-stream 13.0 KB

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2018-02-04 20:05:02 Re: pgAdmin 4 commit: Don't quote variable values used by SET. It's usually
Previous Message Murtuza Zabuawala 2018-02-02 17:46:58 [pgAdmin4][RM#3082] Fix PEP-8 issues