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 |
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 |