calcite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jacques Nadeau <jacq...@apache.org>
Subject Re: Adding new drill functions to optiq syntax
Date Tue, 23 Sep 2014 03:57:19 GMT
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 —
> so I suggest you contribute an Optiq patch. Be sure to add tests to
> SqlOperatorBaseTest along the lines of testVarPopFunc.
>
> Julian
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message