Thanks Jark for checking the doc. hi,Timo, please help to check to see if there is anything else to add.
Best, Forward Jark Wu <imj...@gmail.com> 于2020年1月6日周一 下午2:58写道: > Thanks Forward for the updating. It is much more clearer now about the > returning type, especially JSON_VALUE. > The design doc looks good to me now. > > Best, > Jark > > On Fri, 3 Jan 2020 at 21:42, Forward Xu <forwardxu...@gmail.com> wrote: > > > Hi Timo, Jack, > > Well, I added additional column to describe the return type of each > > function and > > updated the google doc. > > > > Best, > > Forward > > > > Jark Wu <imj...@gmail.com> 于2020年1月3日周五 下午5:48写道: > > > > > Hi Timo, > > > > > > That's a good point. > > > We didn't introduce any new types. We will use the function definition > > > defined by Calcite[1]. > > > So all the functions return STRING/BOOLEAN. > > > > > > Hi Forward, > > > I think we may need an additional column to describe the return type of > > > each function. > > > > > > Best, > > > Jark > > > > > > [1]: > > > > > > > > > https://github.com/apache/calcite/blob/master/core/src/main/java/org/apache/calcite/sql/fun/SqlJsonQueryFunction.java > > > > > > On Fri, 3 Jan 2020 at 17:30, Timo Walther <twal...@apache.org> wrote: > > > > > > > Hi, > > > > > > > > sorry for jumping into the discussion so late. I had a quick look at > > the > > > > FLIP. It looks very nice and detailed. I have one question that I > could > > > > not find in the FLIP itself. Maybe it is hidden in the long > discussion > > > > thread. > > > > > > > > What are the return types of all functions? Do we introduce new types > > > > with this FLIP? Also the RAW types should be avoided. Do all > functions > > > > return STRING/BOOLEAN? > > > > > > > > Thanks, > > > > Timo > > > > > > > > > > > > On 31.12.19 09:39, Hequn Cheng wrote: > > > > > Thanks a lot for the update. +1 to start a vote. > > > > > > > > > > On Tue, Dec 31, 2019 at 2:38 PM Forward Xu <forwardxu...@gmail.com > > > > > > wrote: > > > > > > > > > >> Hi Jark, Hequn, > > > > >> > > > > >> I have updated the documentation. > > > > >> > > > > >> Best, > > > > >> > > > > >> Forward > > > > >> > > > > >> Forward Xu <forwardxu...@gmail.com> 于2019年12月29日周日 下午4:01写道: > > > > >> > > > > >>> Hi Jark, Hequn, > > > > >>> > > > > >>> Thank you very much, Introducing new `TableSymbol`s sounds like a > > > good > > > > >>> idea. +1 for the proposal. > > > > >>> > > > > >>> I think this idea is good, I will add this in the documentation. > > > > >>> > > > > >>> > > > > >>> Best, Forward > > > > >>> > > > > >>> Hequn Cheng <chenghe...@gmail.com> 于2019年12月29日周日 下午3:41写道: > > > > >>> > > > > >>>> Hi Jark, > > > > >>>> > > > > >>>> Introducing new `TableSymbol`s sounds like a good idea. +1 for > the > > > > >>>> proposal. > > > > >>>> @ForwardXu what do you think? Would be great if the document can > > be > > > > >>>> updated > > > > >>>> accordingly. > > > > >>>> > > > > >>>> Best, Hequn > > > > >>>> > > > > >>>> > > > > >>>> On Fri, Dec 27, 2019 at 4:14 PM Jark Wu <imj...@gmail.com> > wrote: > > > > >>>> > > > > >>>>> Thanks for looking into the design Hequn. I agree it would be > > great > > > > to > > > > >>>>> have a full story design. > > > > >>>>> > > > > >>>>> For the ON ERROR and ON EMPTY clause in Table API, some initial > > > > >>>>> thoughts in my mind is that > > > > >>>>> we can introduce some new `TableSymbol`s as the second > parameter > > of > > > > >> json > > > > >>>>> function, e.g. `JsonErrorStrategy`. > > > > >>>>> > > > > >>>>> For example, > > > > >>>>> > > > > >>>>> JSON_VALUE(v, 'lax $.b' ERROR ON ERROR) > > > > >>>>> == is equal to Table API ==> > > > > >>>>> v.jsonValue("lax $.b", JsonErrorStrategy.ERROR) > > > > >>>>> > > > > >>>>> Best, > > > > >>>>> Jark > > > > >>>>> > > > > >>>>> > > > > >>>>> On Thu, 26 Dec 2019 at 23:37, Hequn Cheng < > chenghe...@gmail.com> > > > > >> wrote: > > > > >>>>> > > > > >>>>>> Hi Jark & ForwardXu, > > > > >>>>>> > > > > >>>>>> The design doc looks very nice! Only some minor feedback from > my > > > > >> side. > > > > >>>>>> > > > > >>>>>> As calcite has already implemented the JSON functions, I would > > > > >> suppose > > > > >>>>>> the semantics and implementation are right for SQL. > > > > >>>>>> > > > > >>>>>> For TableAPI, I think the most important is to keep align with > > the > > > > >>>>>> SQL(which has also been mentioned by Jark in the previous > > > > >> discussion). > > > > >>>> Have > > > > >>>>>> an equivalent feature set for all APIs and maintain it > otherwise > > > > >>>> confusion > > > > >>>>>> increases especially when more and more functions are added. > The > > > > >>>> document > > > > >>>>>> has documented how to support TableAPI. I think this is very > > good! > > > > >> And > > > > >>>> it > > > > >>>>>> would be better to also include ON ERROR or ON EMPTY for Table > > > API. > > > > >> We > > > > >>>> can > > > > >>>>>> implement these features step by step, but maybe we should > > design > > > > all > > > > >>>> these > > > > >>>>>> once for all to avoid API changes later. Meanwhile, these > > features > > > > >> are > > > > >>>> also > > > > >>>>>> commonly required by users. > > > > >>>>>> > > > > >>>>>> Would be great to also have your opinions! > > > > >>>>>> > > > > >>>>>> Best, > > > > >>>>>> Hequn > > > > >>>>>> > > > > >>>>>> > > > > >>>>>> On Mon, Dec 23, 2019 at 10:15 AM Jark Wu <imj...@gmail.com> > > > wrote: > > > > >>>>>> > > > > >>>>>>> Hi Forward, > > > > >>>>>>> > > > > >>>>>>> Thanks for creating the FLIP. +1 to start a vote. > > > > >>>>>>> > > > > >>>>>>> @Hequn Cheng <chenghe...@gmail.com> @Kurt Young < > > > > ykt...@gmail.com> > > > > >> , > > > > >>>>>>> could you help to review the design doc too? > > > > >>>>>>> > > > > >>>>>>> Best, > > > > >>>>>>> Jark > > > > >>>>>>> > > > > >>>>>>> > > > > >>>>>>> On Mon, 23 Dec 2019 at 10:10, tison <wander4...@gmail.com> > > > wrote: > > > > >>>>>>> > > > > >>>>>>>> modified: > > > > >>>>>>>> > > > > >>>>>>>> > > > > >>>> > > > > >> > > > > > > > > > > https://lists.apache.org/x/thread.html/b3c0265cc2b660fe11ce550b84a831a7606de12908ff7ff0959a4794@%3Cdev.flink.apache.org%3E > > > > >>>>>>>> > > > > >>>>>>> > > > > >>>> > > > > >>> > > > > >> > > > > > > > > > > > > > > > > > > >