Mailing-List: contact dev-help@optiq.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@optiq.incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of yash360@gmail.com designates
 209.85.215.51 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAFi_UEdE5tKthLyfkkw+riaOdm+kEj6Z+0_Ahj2k=dYD_OJ9Lg@mail.gmail.com>
References: 
 <CAFi_UEeB1GLWiLzTonThqs4KC+OJjboztruK4Wc4obK0ArzP8A@mail.gmail.com>
 <C084EB7F-9EE1-4026-895B-FA05E4F38EC1@gmail.com>
 <CAKa9qDm36o6AjvktNJ1vjFzTAy0G=fd9eB3CKt6BzUVOvW+ysA@mail.gmail.com>
 <CAFi_UEds8hmYzbDzzckmrhtxqiA3BLW-NZzRiDDEcfT-vr+-pw@mail.gmail.com>
 <AC67F773-F5E4-4ABF-9B9C-3A2F4D593A0E@hydromatic.net>
 <CAKa9qDm93RJ0H8znhbEzW8KLph881HOXOm5_MjXkF+E_+xQ4KQ@mail.gmail.com>
 <CAFi_UEeO-LBf+LCMVuv7bA2xL9K5ABASyPyuBnyuzNbTtqbQeg@mail.gmail.com>
 <56CCCF79-31B1-471C-96FB-B75A0322239C@gmail.com>
 <CAFi_UEdE5tKthLyfkkw+riaOdm+kEj6Z+0_Ahj2k=dYD_OJ9Lg@mail.gmail.com>
From: Yash Sharma <yash360@gmail.com>
Date: Tue, 23 Sep 2014 22:29:52 +0530
Message-ID: 
 <CAFi_UEdBvANP4u2EiXsfqMdNa39sYZydi5zo3SuTCGCFFiAYZA@mail.gmail.com>
Subject: Re: Adding new drill functions to optiq syntax
To: "drill-dev@incubator.apache.org" <drill-dev@incubator.apache.org>
Cc: dev@optiq.incubator.apache.org
Content-Type: multipart/alternative; boundary=001a11c346745886ba0503be81bf

--001a11c346745886ba0503be81bf
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

@Julian - I have sent the pull request. Thanks for handling the formatting
and messed up order of keywords.

@Venki - You were right. The JavaCC Error was occurring because the
functions were already declared in CombinedParser.jj.

Just another question - Would it be possible to add keywords to
Registered/Non-Registered keywords via Drill directly? What changes would I
have to do in the Parser.tdd.

Thanks.


On Tue, Sep 23, 2014 at 9:31 PM, Yash Sharma <yash360@gmail.com> wrote:

> @venki/julian: i have got a limited internet connectivity tonight. Will d=
o
> it first thing tomm.
>
> Thanks both.
> On 23/09/2014 9:24 pm, "Julian Hyde" <julianhyde@gmail.com> wrote:
>
>> It looks good. (You messed up the formatting in one place and added
>> out-of-order to alphabetized lists in several places. I have fixed these=
.)
>>
>> Please submit a pull request for your branch and I will commit with my
>> changes.
>>
>> Julian
>>
>> On Sep 23, 2014, at 3:17 AM, Yash Sharma <yash360@gmail.com> wrote:
>>
>> > @Jacques- I would wait for Venki's input. Till then I have created a
>> patch
>> > for Optiq.
>> >
>> > @Julian/Optiq Dev: Could you please review the commit[1] if everything
>> > looks good. Also I am not able to run the test case.
>> > I am using:
>> > $mvn test -Dtest=3DSqlOperatorBaseTest#testStddevPopFunc
>> > It is not able to recognize the test cases.
>> > Works fine with -DfailIfNoTests=3Dflase flag.
>> >
>> > 1:
>> >
>> https://github.com/yssharma/incubator-optiq/commit/9a0b063adf91cee78f3d1=
67c002eb07c6fb7b9b2
>> >
>> > Thanks
>> >
>> >
>> >
>> > On Tue, Sep 23, 2014 at 9:27 AM, Jacques Nadeau <jacques@apache.org>
>> wrote:
>> >
>> >> I believe that Drill also allows addition of reserved words through
>> some of
>> >> the freemarker inclusions but could be mistaken.  I think Venki could
>> >> provide more input if you can't find the spot.
>> >>
>> >> On Mon, Sep 22, 2014 at 12:28 PM, Julian Hyde <julian@hydromatic.net>
>> >> wrote:
>> >>
>> >>> On Sep 22, 2014, at 11:07 AM, Yash Sharma <yash360@gmail.com> wrote:
>> >>>
>> >>>> Am debugging Optiqand DrillOperatorTable.
>> >>>>
>> >>>> Strangely this works with backticks - as pointed out by Kryatal in
>> >>>> DRILL-1441
>> >>>> select `covar_pop`(employee_id, employee_id) FROM cp.`employee.json=
`
>> >>> limit
>> >>>> 10;
>> >>>
>> >>> See my comments on REPLACE in
>> >>> https://issues.apache.org/jira/browse/DRILL-1441. The problems
>> relate to
>> >>> using a non-reserved keyword as an identifier.
>> >>>
>> >>> Regarding COVAR_POP. Probably something similar happening regarding
>> >>> reserved words being. You should probably add it to
>> >> ReservedFunctionName(),
>> >>> just like VAR_POP and STDEV_POP. That is probably tricky to do in
>> Drill =E2=80=94
>> >>> so I suggest you contribute an Optiq patch. Be sure to add tests to
>> >>> SqlOperatorBaseTest along the lines of testVarPopFunc.
>> >>>
>> >>> Julian
>> >>>
>> >>>
>> >>
>>
>>

--001a11c346745886ba0503be81bf--