git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Timestamp as a parameter of UDF


Hello,

I have submitted https://issues.apache.org/jira/browse/CALCITE-2641 . I've
proposed a path there in a comment but I think it would be best to accept
the approach first - then I can contribute the patch.

Thanks for help and support
Pete

On Thu, Oct 25, 2018 at 7:15 PM Julian Hyde <jhyde@xxxxxxxxxx> wrote:

> Two existing JIRA cases that relate to this are
> https://issues.apache.org/jira/browse/CALCITE-1041 <
> https://issues.apache.org/jira/browse/CALCITE-1041> and
> https://issues.apache.org/jira/browse/CALCITE-986 <
> https://issues.apache.org/jira/browse/CALCITE-986>.
>
> This case seems different because it is a user-defined TABLE function. I
> don’t think that has been tested before, therefore I wouldn’t consider it
> “done". The first step is to a bug “DATE and TIMESTAMP arguments to
> user-defined table function”, and the second step is to write a test in
> TableFunctionTest.
>
> Julian
>
>
>
> > On Oct 25, 2018, at 5:32 AM, ptr.bojko@xxxxxxxxx wrote:
> >
> > Will try, thanks.
> >
> > On Thu, Oct 25, 2018 at 2:16 PM Vladimir Sitnikov <
> > sitnikov.vladimir@xxxxxxxxx> wrote:
> >
> >> Then it might be a bug
> >> in org/apache/calcite/linq4j/tree/OptimizeShuttle.java:277
> >> Have you tried to comment that branch out for test purposes?
> >>
> >> Vladimir
> >>
> >
> >
> > --
> > Piotr Bojko
> > http://about.me/ptr.bojko
>
>

-- 
Piotr Bojko
http://about.me/ptr.bojko