| From: | Dave Page <dpage(at)pgadmin(dot)org> |
|---|---|
| To: | Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com> |
| Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
| Subject: | Re: PATCH: FTS Dictionary |
| Date: | 2016-04-14 19:57:28 |
| Message-ID: | CA+OCxozj1SnuXBcrtUKqewmd6TY2odO+JkoUVSJUUx=oBgnfdQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgadmin-hackers |
On Thu, Apr 14, 2016 at 3:41 PM, Sanket Mehta
<sanket(dot)mehta(at)enterprisedb(dot)com> wrote:
> Hi,
>
> PFA patch for FTS Dictionaries.
> Please do review it and let me know in case of any issues.
Thanks - committed with a few cosmetic changes, and a couple of minor
issues to resolve:
1) The template is not shown in properties views if it is not in a
system schema (because of the way you are dealing with schema names
etc). Please also check how the existing code will behave, if there is
a template in the public schema with the same name as one in
pg_catalog.
2) Please strip the quotes from the options before displaying them.
I'll create a card for this.
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Sanket Mehta | 2016-04-14 19:59:01 | Re: PATCH: FTS Dictionary |
| Previous Message | Dave Page | 2016-04-14 19:54:18 | pgAdmin 4 commit: FTS Dictionary support (icons) |