Thanks, applied.
On Tue, Apr 10, 2018 at 6:57 PM, Murtuza Zabuawala <
murtuza.zabuaw...@enterprisedb.com> wrote:
> Thanks for reviewing Joao.
>
> Yeah missed to run tests. I assumed its just a SQL formatting it wouldn't
> hurt anything but I was wrong :)
> Here's updated patch.
>
> --
> Regards,
Thanks for reviewing Joao.
Yeah missed to run tests. I assumed its just a SQL formatting it wouldn't
hurt anything but I was wrong :)
Here's updated patch.
--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
On Tue, Apr 10, 2018 at 8:18 PM,
Hi Murtuza,
Looks like the tests are failing due to this change.
I also got some strange result on the linter, but I believe it is something
related to this machine.
Thanks
Joao
On Tue, Apr 10, 2018 at 6:20 AM Murtuza Zabuawala <
murtuza.zabuaw...@enterprisedb.com> wrote:
> Hi Dave,
>
> A minor
Hi Dave,
A minor fix for explain functionality.
*Issue:* Due to multiline explain options in the SQL template CodeMirror
was failing to highlight the exact line of error.
--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
RM_3257.diff
Des