From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | George Gelashvili <ggelashvili(at)pivotal(dot)io> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
Subject: | Re: [Patch] Refactor sql template version picking |
Date: | 2017-01-27 16:22:38 |
Message-ID: | CA+OCxoygd7LbWfZwGpULLHYrmFLaetVozZfJp2YFziv4FZuG+Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Very nice indeed! I didn't realise we'd ended up with quite so many
duplicated templates.
Both patches look good to me - really the only thing that caught my
eye was the name versioned_template_loader which is somewhat longer
than I'd prefer.
As it's a major change, and we're going to be wrapping 1.2 in a little
over a week, I'd like some further review before committing. Khushboo,
can you take a look first thing on Monday please? If you're happy,
I'll commit and ask Fahar to do some testing.
Thanks!
On Thu, Jan 26, 2017 at 8:48 PM, George Gelashvili
<ggelashvili(at)pivotal(dot)io> wrote:
> As a followup, we created this additional patch removing now-redundant sql
> templates. If you're interested in how we decided what files to delete, the
> script we used is here. We only removed files that were exactly identical
> between multiple postgres versions for the same feature.
>
> This patch should be applied on a master patched with
> version-aware-sql-template-loader-refactor.diff
>
>
> On Thu, Jan 26, 2017 at 12:22 PM, George Gelashvili <ggelashvili(at)pivotal(dot)io>
> wrote:
>>
>> Hi Hackers!
>>
>> Since we are preparing to add greenplum support, we made a new template
>> loader to automatically pick the available sql template file that
>> corresponds to the postgres version number.
>>
>> Our next patch will be to remove duplicated sql template files since many
>> of the files are identical between versions.
>>
>> Cheers,
>> George & Tira
>
>
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2017-01-27 16:27:07 | Re: [Patch] Refactor sql template version picking |
Previous Message | Dave Page | 2017-01-27 16:11:12 | Re: Acceptance Tests against a browser (WIP) |