public inbox for [email protected]
help / color / mirror / Atom feedFrom: Sanket Mehta <[email protected]>
To: Ashesh Vashi <[email protected]>
Cc: Harshal Dhumal <[email protected]>
Cc: Akshay Joshi <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: PATCH: FTS configuration node
Date: Wed, 18 May 2016 12:52:37 +0530
Message-ID: <CA+yw=mMMA=R6dMSAnP3GWH8RBw+JO2JwVVC96r72JDNCLfDAYA@mail.gmail.com> (raw)
In-Reply-To: <CAG7mmox_mZy+FATLMhssRJMGd1UfNJ-8mN2dcVaaHqG=UTPPGw@mail.gmail.com>
References: <CA+yw=mN+xtYDw_E-H2jexDtrY60kb=V5bMwDWBmm8PBDSoSnfw@mail.gmail.com>
<CAFiP3vx=_Tm5c5pYYkU01LEV=rV9mUTgZjdLU6fPaqehhwF1=Q@mail.gmail.com>
<CA+yw=mMENphfZiQeU39Th7+E5aOK2eUpRmg5OdomV1RCzouE_w@mail.gmail.com>
<CA+yw=mM1X5E+4W=vD19HBjaCvb0dQSzCU3nw4CksrvHtRqd0MQ@mail.gmail.com>
<CANxoLDcAYhKC2TH_ZGwioTRYQY7Da+Jkq__-OrGWJ958WddQ2A@mail.gmail.com>
<CA+yw=mPhxm_hcWTJ1LWZttJSva7YNf-wRD0_HUri81L3Udz8Gg@mail.gmail.com>
<CAFiP3vzwOVU1F_kZ5YzcXE0f36UvYTviV6Xjt5osWz=Xe0+9DQ@mail.gmail.com>
<CA+yw=mP4jxMTjAUYwGPYrkjc3H1NOmXHOzPH_nf6NFci0rq6bw@mail.gmail.com>
<CAG7mmox_mZy+FATLMhssRJMGd1UfNJ-8mN2dcVaaHqG=UTPPGw@mail.gmail.com>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgadmin-hackers>
On Wed, May 18, 2016 at 12:42 PM, Ashesh Vashi <
[email protected]> wrote:
> On Wed, May 18, 2016 at 12:41 PM, Sanket Mehta <
> [email protected]> wrote:
>
>> Hi,
>>
>> PFA the revised patch.
>> Response is lnline.
>>
>> Regards,
>> Sanket Mehta
>> Sr Software engineer
>> Enterprisedb
>>
>> On Tue, May 17, 2016 at 12:54 PM, Harshal Dhumal <
>> [email protected]> wrote:
>>
>>> Hi Sanket,
>>>
>>> Please find my review comments below:
>>>
>>> 1. In create mode it generates wrong sql.
>>>
>>> CREATE TEXT SEARCH CONFIGURATION test.asdf (
>>> COPY=
>>> );
>>>
>>> steps to reproduce:
>>> a] Fill up any necessary fields in general tab.
>>> b] On definition tab set Copy config filed and navigate to SQL tab.
>>> c] Come back again to definition tab clear Copy config field and set
>>> Parser field and again navigate to SQL tab
>>>
>> Fixed
>>
>>>
>>> 2. Create new FTS configuration with parser throws below error.
>>>
>>> provide atleast copy config or parser.
>>>
>>> (this is because you have used key name for parser in js as "prsname"
>>> and in python you have applied validations on key "parser")
>>>
>> Fixed
>>
>>>
>>> 3. When we clear dictionary for any token using cross (x) button in
>>> select2; JavaScript code for selecct2 fails. (see attached screenshot)
>>>
>> This occurs due to issue in select2 control. Its a generalized issue
>> which will be resolved later.
>>
> When you say - it is common issue, can you please point where else you've
> seen this behaviour?
>
The same behaviour occurs in all nodes where multiselect cell is being used
like, exclusion control, debugger etc.
>
> --
>
> Thanks & Regards,
>
> Ashesh Vashi
> EnterpriseDB INDIA: Enterprise PostgreSQL Company
> <http://www.enterprisedb.com/;
>
>
> *http://www.linkedin.com/in/asheshvashi*
> <http://www.linkedin.com/in/asheshvashi;
>
>>
>>>
>>>
>>>
>>>
>>>
>>> --
>>> *Harshal Dhumal*
>>> *Software Engineer *
>>>
>>>
>>>
>>> EenterpriseDB <http://www.enterprisedb.com;
>>>
>>> On Mon, May 16, 2016 at 7:37 PM, Sanket Mehta <
>>> [email protected]> wrote:
>>>
>>>> Hi,
>>>>
>>>> Revised patch is attached with this mail.
>>>> My response is inline.
>>>>
>>>>
>>>> Regards,
>>>> Sanket Mehta
>>>> Sr Software engineer
>>>> Enterprisedb
>>>>
>>>> On Fri, May 13, 2016 at 6:20 PM, Akshay Joshi <
>>>> [email protected]> wrote:
>>>>
>>>>> Hi Sanket
>>>>>
>>>>> Below are my review comments:
>>>>>
>>>>>
>>>>> - Add button should be disabled in Tokens tab while creating.
>>>>>
>>>>> Done
>>>>
>>>>>
>>>>> - Unable to click on down arrow for token select control.
>>>>>
>>>>> Done
>>>>
>>>>>
>>>>> - Title should be change from "Create FTS-dictionaries" to "Create
>>>>> FTS-Configuration".
>>>>>
>>>>> Done
>>>>
>>>>>
>>>>> - Tree node is not getting refreshed on name change.
>>>>>
>>>>> Done
>>>>
>>>>>
>>>>> - Unable to rename FTS Configuration when name contains any
>>>>> capital letter.
>>>>>
>>>>> Done
>>>>
>>>>>
>>>>> - If user tries to add already existing token then respective row
>>>>> in the backgrid should be highlighted.
>>>>>
>>>>> Done
>>>>
>>>>>
>>>>> - "URL not found" when click on SQL help icon from the properties
>>>>> dialog.
>>>>>
>>>>> Done
>>>>
>>>>>
>>>>> - SQL should not be generated when remove and add the same token
>>>>> again. Currently it is creating two sql's one for remove and
>>>>> another is for add.
>>>>> - *Fix for the above issue is*: - Add "keys: ['token']," to
>>>>> TokenModel in your js file.
>>>>>
>>>>> Done
>>>>
>>>>
>>>>>
>>>>> On Fri, May 13, 2016 at 2:58 PM, Sanket Mehta <
>>>>> [email protected]> wrote:
>>>>>
>>>>>> Hi All,
>>>>>>
>>>>>> Last patch for FTS configuration does not contain node.ui.js file
>>>>>> Kindly ignore it.
>>>>>>
>>>>>> Here is the new revised patch attached with this mail.
>>>>>> Please do review it and let me know if any changes required
>>>>>>
>>>>>> Regards,
>>>>>> Sanket Mehta
>>>>>> Sr Software engineer
>>>>>> Enterprisedb
>>>>>>
>>>>>> On Thu, May 12, 2016 at 4:38 PM, Sanket Mehta <
>>>>>> [email protected]> wrote:
>>>>>>
>>>>>>> Hi,
>>>>>>>
>>>>>>> PFA the revised patch.
>>>>>>> Please do review it and let me know if anything is not proper.
>>>>>>>
>>>>>>> Regards,
>>>>>>> Sanket Mehta
>>>>>>> Sr Software engineer
>>>>>>> Enterprisedb
>>>>>>>
>>>>>>> On Thu, May 5, 2016 at 8:19 PM, Harshal Dhumal <
>>>>>>> [email protected]> wrote:
>>>>>>>
>>>>>>>> + patch link
>>>>>>>>
>>>>>>>>
>>>>>>>> http://www.postgresql.org/message-id/[email protected]....
>>>>>>>>
>>>>>>>> --
>>>>>>>> *Harshal Dhumal*
>>>>>>>> *Software Engineer *
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> EenterpriseDB <http://www.enterprisedb.com;
>>>>>>>>
>>>>>>>> On Thu, May 5, 2016 at 8:18 PM, Sanket Mehta <
>>>>>>>> [email protected]> wrote:
>>>>>>>>
>>>>>>>>> Hi,
>>>>>>>>>
>>>>>>>>> PFA first patch for FTS configuration node.
>>>>>>>>>
>>>>>>>>> It depends upon backgrid select2cell multi select control, for
>>>>>>>>> which Harshal has sent the patch recently.
>>>>>>>>> Please do apply his patch first and then apply this patch.
>>>>>>>>>
>>>>>>>>> Please do review it and let me know if any changes are required.
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Regards,
>>>>>>>>> Sanket Mehta
>>>>>>>>> Sr Software engineer
>>>>>>>>> Enterprisedb
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> --
>>>>>>>>> Sent via pgadmin-hackers mailing list (
>>>>>>>>> [email protected])
>>>>>>>>> To make changes to your subscription:
>>>>>>>>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Sent via pgadmin-hackers mailing list ([email protected]
>>>>>> )
>>>>>> To make changes to your subscription:
>>>>>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> *Akshay Joshi*
>>>>> *Principal Software Engineer *
>>>>>
>>>>>
>>>>>
>>>>> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> Sent via pgadmin-hackers mailing list ([email protected])
>>>> To make changes to your subscription:
>>>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>>>
>>>>
>>>
>>
>>
>> --
>> Sent via pgadmin-hackers mailing list ([email protected])
>> To make changes to your subscription:
>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>
>>
>
view thread (12+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected]
Subject: Re: PATCH: FTS configuration node
In-Reply-To: <CA+yw=mMMA=R6dMSAnP3GWH8RBw+JO2JwVVC96r72JDNCLfDAYA@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox