Hi Yubin, Sorry, please ignore my last reply (wrong context). I also asked Leonard, your proposal to extend the `CatalogDescriptor` should be okay.
Thank you for your update : ) ! Best, Lincoln Lee Lincoln Lee <lincoln.8...@gmail.com> 于2024年3月20日周三 09:35写道: > Hi Yubin, > > Thank you for detailed explaination! I overlooked `CatalogBaseTable`, in > fact > there is already a `String getComment();` interface similar to `database` > and `table`. > Can we continue the work on FLINK-21665 and complete its implementation? > It seems to be very close. > > Best, > Lincoln Lee > > > Yubin Li <lyb5...@gmail.com> 于2024年3月20日周三 01:42写道: > >> Hi Lincoln, >> >> Thanks for your detailed comments! >> >> Supporting comments for `Catalog` is a really helpful feature, I agree >> with you to make it introduced in this FLIP, thank you for pointing >> that out :) >> >> Concerning the implementation, I propose to introduce `getComment()` >> method in `CatalogDescriptor`, and the reasons are as follows. WDYT? >> 1. For the sake of design consistency, follow the design of FLIP-295 >> [1] which introduced `CatalogStore` component, `CatalogDescriptor` >> includes names and attributes, both of which are used to describe the >> catalog, and `comment` can be added smoothly. >> 2. Extending the existing class rather than add new method to the >> existing interface, Especially, the `Catalog` interface, as a core >> interface, is used by a series of important components such as >> `CatalogFactory`, `CatalogManager` and `FactoryUtil`, and is >> implemented by a large number of connectors such as JDBC, Paimon, and >> Hive. Adding methods to it will greatly increase the implementation >> complexity, and more importantly, increase the cost of iteration, >> maintenance, and verification. >> >> Please see FLIP doc [2] for details. >> >> [1] >> https://cwiki.apache.org/confluence/display/FLINK/FLIP-295%3A+Support+lazy+initialization+of+catalogs+and+persistence+of+catalog+configurations >> [2] >> https://cwiki.apache.org/confluence/display/FLINK/FLIP-436%3A+Introduce+Catalog-related+Syntax >> >> Best, >> Yubin >> >> On Tue, Mar 19, 2024 at 9:57 PM Lincoln Lee <lincoln.8...@gmail.com> >> wrote: >> > >> > Hi Yubin, >> > >> > Thanks for your quickly response! >> > >> > It would be better to support comments just like create `database` and >> > `table` with comment. >> > That is, add `String getComment();` to the current `Catalog` interface. >> > WDYT? >> > >> > Best, >> > Lincoln Lee >> > >> > >> > Yubin Li <lyb5...@gmail.com> 于2024年3月19日周二 21:44写道: >> > >> > > Hi Lincoln, >> > > >> > > Good catch. Thanks for your suggestions. >> > > >> > > I found that the creation statements of database and table both >> > > support specifying "if not exists". For the sake of syntactic >> > > consistency and user practicality, We could introduce the '[if not >> > > exists]' clause to the 'create catalog' statement. >> > > >> > > As for the introduction of the `catalog comment` feature, it may >> > > involve changes to the Catalog structure, which can be left for future >> > > discussion. >> > > >> > > WDYT? Looking forward to your feedback :) >> > > >> > > Best, >> > > Yubin >> > > >> > > On Tue, Mar 19, 2024 at 9:06 PM Lincoln Lee <lincoln.8...@gmail.com> >> > > wrote: >> > > > >> > > > Hi Yubin, >> > > > >> > > > Big +1 for completing the catalog api! >> > > > There's a minor addition[1] which does not affect the vote could >> also be >> > > > considered. >> > > > >> > > > [1] https://issues.apache.org/jira/browse/FLINK-21665 >> > > > >> > > > >> > > > Best, >> > > > Lincoln Lee >> > > > >> > > > >> > > > Yubin Li <lyb5...@gmail.com> 于2024年3月18日周一 17:44写道: >> > > > >> > > > > Hi Jark, >> > > > > >> > > > > Thanks for your response, I have updated FLIP-436: Introduce >> > > > > Catalog-related Syntax [1] as you suggested. >> > > > > >> > > > > If there are no more comments within 24 hours, I will start a >> vote for >> > > > > this, thanks :) >> > > > > >> > > > > Best, >> > > > > Yubin >> > > > > >> > > > > [1] >> > > > > >> > > >> https://cwiki.apache.org/confluence/display/FLINK/FLIP-436%3A+Introduce+Catalog-related+Syntax >> > > > > >> > > > > On Mon, Mar 18, 2024 at 4:39 PM Jark Wu <imj...@gmail.com> wrote: >> > > > > > >> > > > > > Hi Yubin, >> > > > > > >> > > > > > Thanks for the quick response. The suggestion sounds good to me! >> > > > > > >> > > > > > Best, >> > > > > > Jark >> > > > > > >> > > > > > On Mon, 18 Mar 2024 at 13:06, Yubin Li <lyb5...@gmail.com> >> wrote: >> > > > > > >> > > > > > > Hi Jark, >> > > > > > > >> > > > > > > Good pointing! Thanks for your reply, there are some details >> to >> > > align >> > > > > :) >> > > > > > > >> > > > > > > 1. I think the purpose of DESCRIBE CATALOG is to display >> metadata >> > > > > > > > information including catalog name, >> > > > > > > > catalog comment (may be introduced in the future), catalog >> type, >> > > and >> > > > > > > > catalog properties (for example [1]) >> > > > > > > >> > > > > > > Adopting { DESC | DESCRIBE } CATALOG [ EXTENDED ] xx as formal >> > > syntax, >> > > > > > > Producing rich and compatible results for future needs is very >> > > > > important. >> > > > > > > When >> > > > > > > specifying "extended" in the syntax, it will output the >> complete >> > > > > > > information including >> > > > > > > properties.The complete output example is as follows: >> > > > > > > >> > > > > > > >> > > > > >> > > >> +---------------------------------+---------------------------------------------------+ >> > > > > > > | catalog_description_item | catalog_description_value >> > > > > | >> > > > > > > >> > > > > > > >> > > > > >> > > >> +---------------------------------+---------------------------------------------------+ >> > > > > > > | Name | cat1 >> > > > > > > | >> > > > > > > | Type | >> > > generic_in_memory >> > > > > > > | >> > > > > > > | Comment | >> > > > > > > | >> > > > > > > | Properties | ((k1,v1), >> > > (k2,v2)) >> > > > > > > | >> > > > > > > >> > > > > > > >> > > > > >> > > >> +---------------------------------+---------------------------------------------------+ >> > > > > > > >> > > > > > > 2. Could you add support for ALTER CATALOG xxx UNSET >> ('mykey')? >> > > This is >> > > > > > > > also very useful in ALTER TABLE. >> > > > > > > >> > > > > > > I found that there is already an ALTER TABLE xxx RESET >> ('mykey') >> > > > > syntax [1] >> > > > > > > now, >> > > > > > > which will reset the myKey attribute of a certain table to the >> > > default >> > > > > > > value. For catalogs, >> > > > > > > it might be better to use ALTER CATALOG xxx RESET ('mykey') >> for the >> > > > > sake of >> > > > > > > design >> > > > > > > consistency. >> > > > > > > >> > > > > > > WDYT? Looking forward to your suggestions. >> > > > > > > >> > > > > > > Best, >> > > > > > > Yubin >> > > > > > > >> > > > > > > [1] >> > > > > > > >> > > > > > > >> > > > > >> > > >> https://nightlies.apache.org/flink/flink-docs-master/docs/dev/table/sql/alter/#reset >> > > > > > > >> > > > > > > >> > > > > > > On Mon, Mar 18, 2024 at 11:49 AM Jark Wu <imj...@gmail.com> >> wrote: >> > > > > > > >> > > > > > > > Hi Yubin, >> > > > > > > > >> > > > > > > > Thanks for updating the FLIP. The updated version looks >> good in >> > > > > general. >> > > > > > > > I only have 2 minor comments. >> > > > > > > > >> > > > > > > > 1. I think the purpose of DESCRIBE CATALOG is to display >> metadata >> > > > > > > > information including catalog name, >> > > > > > > > catalog comment (may be introduced in the future), catalog >> type, >> > > and >> > > > > > > > catalog properties (for example [1]). >> > > > > > > > Expanding all properties may limit this syntax to include >> more >> > > > > metadata >> > > > > > > > information in the future. >> > > > > > > > >> > > > > > > > 2. Could you add support for ALTER CATALOG xxx UNSET >> ('mykey')? >> > > This >> > > > > is >> > > > > > > > also very useful in ALTER TABLE. >> > > > > > > > >> > > > > > > > Best, >> > > > > > > > Jark >> > > > > > > > >> > > > > > > > [1]: >> > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> https://docs.databricks.com/en/sql/language-manual/sql-ref-syntax-aux-describe-schema.html >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > > > On Fri, 15 Mar 2024 at 12:06, Yubin Li <lyb5...@gmail.com> >> > > wrote: >> > > > > > > > >> > > > > > > > > Hi Xuyang, >> > > > > > > > > >> > > > > > > > > Thank you for pointing this out, The parser part of >> `describe >> > > > > catalog` >> > > > > > > > > syntax >> > > > > > > > > has indeed been implemented in FLIP-69, and it is not >> actually >> > > > > > > available. >> > > > > > > > > we can complete the syntax in this FLIP [1]. I have >> updated >> > > the >> > > > > doc :) >> > > > > > > > > >> > > > > > > > > Best, >> > > > > > > > > Yubin >> > > > > > > > > >> > > > > > > > > [1] >> > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> https://cwiki.apache.org/confluence/display/FLINK/FLIP-436%3A+Introduce+Catalog-related+Syntax >> > > > > > > > > >> > > > > > > > > On Fri, Mar 15, 2024 at 10:12 AM Xuyang < >> xyzhong...@163.com> >> > > > > wrote: >> > > > > > > > > >> > > > > > > > > > Hi, Yubin. Big +1 for this Flip. I just left one minor >> > > comment >> > > > > > > > following. >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > I found that although flink has not supported syntax >> > > 'DESCRIBE >> > > > > > > CATALOG >> > > > > > > > > > catalog_name' currently, it was already >> > > > > > > > > > discussed in flip-69[1], do we need to restart >> discussing it? >> > > > > > > > > > I don't have a particular preference regarding the >> restart >> > > > > > > discussion. >> > > > > > > > It >> > > > > > > > > > seems that there is no difference on this syntax >> > > > > > > > > > in FLIP-436, so maybe it would be best to refer back to >> > > FLIP-69 >> > > > > in >> > > > > > > this >> > > > > > > > > > FLIP. WDYT? >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > [1] >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> https://cwiki.apache.org/confluence/display/FLINK/FLIP-69%3A+Flink+SQL+DDL+Enhancement >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > -- >> > > > > > > > > > >> > > > > > > > > > Best! >> > > > > > > > > > Xuyang >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > At 2024-03-15 02:49:59, "Yubin Li" <lyb5...@gmail.com> >> > > wrote: >> > > > > > > > > > >Hi folks, >> > > > > > > > > > > >> > > > > > > > > > >Thank you all for your input, it really makes sense to >> > > introduce >> > > > > > > > missing >> > > > > > > > > > >catalog-related SQL syntaxes under this FLIP, and I >> have >> > > > > changed the >> > > > > > > > > > >title of doc to "FLIP-436: Introduce Catalog-related >> > > Syntax". >> > > > > > > > > > > >> > > > > > > > > > >After comprehensive consideration, the following >> syntaxes >> > > > > should be >> > > > > > > > > > >introduced, more suggestions are welcome :) >> > > > > > > > > > > >> > > > > > > > > > >> 1. SHOW CREATE CATALOG catalog_name >> > > > > > > > > > >> 2. DESCRIBE/DESC CATALOG catalog_name >> > > > > > > > > > >> 3. ALTER CATALOG catalog_name SET (key1=val1, >> key2=val2, >> > > ...) >> > > > > > > > > > > >> > > > > > > > > > >Regarding the `alter catalog` syntax format, I refer >> to the >> > > > > current >> > > > > > > > > design >> > > > > > > > > > >of `alter database`. >> > > > > > > > > > > >> > > > > > > > > > >Given that CatalogManager already provides catalog >> > > operations >> > > > > such >> > > > > > > as >> > > > > > > > > > >create, get, and unregister, and in order to facilitate >> > > future >> > > > > > > > > > >implementation >> > > > > > > > > > >of audit tracking, I propose to introduce the >> alterCatalog() >> > > > > > > function >> > > > > > > > in >> > > > > > > > > > >CatalogManager. WDYT? >> > > > > > > > > > > >> > > > > > > > > > >Please see details in FLIP doc [1] . >> > > > > > > > > > > >> > > > > > > > > > >Best, >> > > > > > > > > > >Yubin >> > > > > > > > > > > >> > > > > > > > > > >[1] >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> https://cwiki.apache.org/confluence/display/FLINK/FLIP-436%3A+Introduce+Catalog-related+Syntax >> > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > >On Thu, Mar 14, 2024 at 11:07 PM Leonard Xu < >> > > xbjt...@gmail.com> >> > > > > > > > wrote: >> > > > > > > > > > > >> > > > > > > > > > >> Hi Yubin, >> > > > > > > > > > >> >> > > > > > > > > > >> Thanks for driving the discussion, generally +1 for >> the >> > > FLIP, >> > > > > big >> > > > > > > +1 >> > > > > > > > > to >> > > > > > > > > > >> finalize the whole catalog syntax story in one FLIP, >> > > > > > > > > > >> thus I want to jump into the discussion again after >> you >> > > > > completed >> > > > > > > > the >> > > > > > > > > > >> whole catalog syntax story. >> > > > > > > > > > >> >> > > > > > > > > > >> Best, >> > > > > > > > > > >> Leonard >> > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > > > > >> > 2024年3月14日 下午8:39,Roc Marshal <flin...@126.com> >> 写道: >> > > > > > > > > > >> > >> > > > > > > > > > >> > Hi, Yubin >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > Thank you for initiating this discussion! +1 for >> the >> > > > > proposal. >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > Best, >> > > > > > > > > > >> > Yuepeng Pan >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > > >> > At 2024-03-14 18:57:35, "Ferenc Csaky" >> > > > > > > <ferenc.cs...@pm.me.INVALID >> > > > > > > > > >> > > > > > > > > > >> wrote: >> > > > > > > > > > >> >> Hi Yubin, >> > > > > > > > > > >> >> >> > > > > > > > > > >> >> Thank you for initiating this discussion! +1 for >> the >> > > > > proposal. >> > > > > > > > > > >> >> >> > > > > > > > > > >> >> I also think it makes sense to group the missing >> > > catalog >> > > > > > > related >> > > > > > > > > > >> >> SQL syntaxes under this FLIP. >> > > > > > > > > > >> >> >> > > > > > > > > > >> >> Looking forward to these features! >> > > > > > > > > > >> >> >> > > > > > > > > > >> >> Best, >> > > > > > > > > > >> >> Ferenc >> > > > > > > > > > >> >> >> > > > > > > > > > >> >> >> > > > > > > > > > >> >> >> > > > > > > > > > >> >> >> > > > > > > > > > >> >> On Thursday, March 14th, 2024 at 08:31, Jane Chan >> < >> > > > > > > > > > >> qingyue....@gmail.com> wrote: >> > > > > > > > > > >> >> >> > > > > > > > > > >> >>> >> > > > > > > > > > >> >>> >> > > > > > > > > > >> >>> Hi Yubin, >> > > > > > > > > > >> >>> >> > > > > > > > > > >> >>> Thanks for leading the discussion. I'm +1 for the >> > > FLIP. >> > > > > > > > > > >> >>> >> > > > > > > > > > >> >>> As Jark said, it's a good opportunity to enhance >> the >> > > > > syntax >> > > > > > > for >> > > > > > > > > > Catalog >> > > > > > > > > > >> >>> from a more comprehensive perspective. So, I >> suggest >> > > > > expanding >> > > > > > > > the >> > > > > > > > > > >> scope of >> > > > > > > > > > >> >>> this FLIP by focusing on the mechanism instead >> of one >> > > use >> > > > > case >> > > > > > > > to >> > > > > > > > > > >> enhance >> > > > > > > > > > >> >>> the overall functionality. WDYT? >> > > > > > > > > > >> >>> >> > > > > > > > > > >> >>> Best, >> > > > > > > > > > >> >>> Jane >> > > > > > > > > > >> >>> >> > > > > > > > > > >> >>> On Thu, Mar 14, 2024 at 11:38 AM Hang Ruan >> > > > > > > > ruanhang1...@gmail.com >> > > > > > > > > > >> wrote: >> > > > > > > > > > >> >>> >> > > > > > > > > > >> >>>> Hi, Yubin. >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>> Thanks for the FLIP. +1 for it. >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>> Best, >> > > > > > > > > > >> >>>> Hang >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>> Yubin Li lyb5...@gmail.com 于2024年3月14日周四 >> 10:15写道: >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>>> Hi Jingsong, Feng, and Jeyhun >> > > > > > > > > > >> >>>>> >> > > > > > > > > > >> >>>>> Thanks for your support and feedback! >> > > > > > > > > > >> >>>>> >> > > > > > > > > > >> >>>>>> However, could we add a new method >> > > > > `getCatalogDescriptor()` >> > > > > > > > to >> > > > > > > > > > >> >>>>>> CatalogManager instead of directly exposing >> > > > > CatalogStore? >> > > > > > > > > > >> >>>>> >> > > > > > > > > > >> >>>>> Good point, Besides the audit tracking issue, >> The >> > > > > proposed >> > > > > > > > > feature >> > > > > > > > > > >> >>>>> only requires `getCatalogDescriptor()` >> function. >> > > > > Exposing >> > > > > > > > > > components >> > > > > > > > > > >> >>>>> with excessive functionality will bring >> unnecessary >> > > > > risks, I >> > > > > > > > > have >> > > > > > > > > > >> made >> > > > > > > > > > >> >>>>> modifications in the FLIP doc [1]. Thank Feng >> :) >> > > > > > > > > > >> >>>>> >> > > > > > > > > > >> >>>>>> Showing the SQL parser implementation in the >> FLIP >> > > for >> > > > > the >> > > > > > > SQL >> > > > > > > > > > syntax >> > > > > > > > > > >> >>>>>> might be a bit confusing. Also, the formal >> > > definition >> > > > > is >> > > > > > > > > missing >> > > > > > > > > > for >> > > > > > > > > > >> >>>>>> this SQL clause. >> > > > > > > > > > >> >>>>> >> > > > > > > > > > >> >>>>> Thank Jeyhun for pointing it out :) I have >> updated >> > > the >> > > > > doc >> > > > > > > > [1] . >> > > > > > > > > > >> >>>>> >> > > > > > > > > > >> >>>>> [1] >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=296290756 >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>>> Best, >> > > > > > > > > > >> >>>>> Yubin >> > > > > > > > > > >> >>>>> >> > > > > > > > > > >> >>>>> On Thu, Mar 14, 2024 at 2:18 AM Jeyhun Karimov >> > > > > > > > > > je.kari...@gmail.com >> > > > > > > > > > >> >>>>> wrote: >> > > > > > > > > > >> >>>>> >> > > > > > > > > > >> >>>>>> Hi Yubin, >> > > > > > > > > > >> >>>>>> >> > > > > > > > > > >> >>>>>> Thanks for the proposal. +1 for it. >> > > > > > > > > > >> >>>>>> I have one comment: >> > > > > > > > > > >> >>>>>> >> > > > > > > > > > >> >>>>>> I would like to see the SQL syntax for the >> proposed >> > > > > > > > statement. >> > > > > > > > > > >> Showing >> > > > > > > > > > >> >>>>>> the >> > > > > > > > > > >> >>>>>> SQL parser implementation in the FLIP >> > > > > > > > > > >> >>>>>> for the SQL syntax might be a bit confusing. >> Also, >> > > the >> > > > > > > formal >> > > > > > > > > > >> >>>>>> definition >> > > > > > > > > > >> >>>>>> is >> > > > > > > > > > >> >>>>>> missing for this SQL clause. >> > > > > > > > > > >> >>>>>> Maybe something like [1] might be useful. >> WDYT? >> > > > > > > > > > >> >>>>>> >> > > > > > > > > > >> >>>>>> Regards, >> > > > > > > > > > >> >>>>>> Jeyhun >> > > > > > > > > > >> >>>>>> >> > > > > > > > > > >> >>>>>> [1] >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> https://github.com/apache/flink/blob/0da60ca1a4754f858cf7c52dd4f0c97ae0e1b0cb/docs/content/docs/dev/table/sql/show.md?plain=1#L620-L632 >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>>>> On Wed, Mar 13, 2024 at 3:28 PM Feng Jin >> > > > > > > > jinfeng1...@gmail.com >> > > > > > > > > > >> >>>>>> wrote: >> > > > > > > > > > >> >>>>>> >> > > > > > > > > > >> >>>>>>> Hi Yubin >> > > > > > > > > > >> >>>>>>> >> > > > > > > > > > >> >>>>>>> Thank you for initiating this FLIP. >> > > > > > > > > > >> >>>>>>> >> > > > > > > > > > >> >>>>>>> I have just one minor question: >> > > > > > > > > > >> >>>>>>> >> > > > > > > > > > >> >>>>>>> I noticed that we added a new function >> > > > > `getCatalogStore` >> > > > > > > to >> > > > > > > > > > expose >> > > > > > > > > > >> >>>>>>> CatalogStore, and it seems fine. >> > > > > > > > > > >> >>>>>>> However, could we add a new method >> > > > > > > `getCatalogDescriptor()` >> > > > > > > > to >> > > > > > > > > > >> >>>>>>> CatalogManager instead of directly exposing >> > > > > CatalogStore? >> > > > > > > > > > >> >>>>>>> By only providing the >> `getCatalogDescriptor()` >> > > > > interface, >> > > > > > > it >> > > > > > > > > > may be >> > > > > > > > > > >> >>>>>>> easier >> > > > > > > > > > >> >>>>>>> for us to implement audit tracking in >> > > CatalogManager >> > > > > in >> > > > > > > the >> > > > > > > > > > future. >> > > > > > > > > > >> >>>>>>> WDYT ? >> > > > > > > > > > >> >>>>>>> Although we have only collected some modified >> > > events >> > > > > at >> > > > > > > the >> > > > > > > > > > >> >>>>>>> moment.[1] >> > > > > > > > > > >> >>>>>>> >> > > > > > > > > > >> >>>>>>> [1]. >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> https://cwiki.apache.org/confluence/display/FLINK/FLIP-294%3A+Support+Customized+Catalog+Modification+Listener >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>>>>> Best, >> > > > > > > > > > >> >>>>>>> Feng >> > > > > > > > > > >> >>>>>>> >> > > > > > > > > > >> >>>>>>> On Wed, Mar 13, 2024 at 5:31 PM Jingsong Li >> > > > > > > > > > jingsongl...@gmail.com >> > > > > > > > > > >> >>>>>>> wrote: >> > > > > > > > > > >> >>>>>>> >> > > > > > > > > > >> >>>>>>>> +1 for this. >> > > > > > > > > > >> >>>>>>>> >> > > > > > > > > > >> >>>>>>>> We are missing a series of catalog related >> > > syntaxes. >> > > > > > > > > > >> >>>>>>>> Especially after the introduction of catalog >> > > store. >> > > > > [1] >> > > > > > > > > > >> >>>>>>>> >> > > > > > > > > > >> >>>>>>>> [1] >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> https://cwiki.apache.org/confluence/display/FLINK/FLIP-295%3A+Support+lazy+initialization+of+catalogs+and+persistence+of+catalog+configurations >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>>>>>> Best, >> > > > > > > > > > >> >>>>>>>> Jingsong >> > > > > > > > > > >> >>>>>>>> >> > > > > > > > > > >> >>>>>>>> On Wed, Mar 13, 2024 at 5:09 PM Yubin Li >> > > > > > > lyb5...@gmail.com >> > > > > > > > > > >> >>>>>>>> wrote: >> > > > > > > > > > >> >>>>>>>> >> > > > > > > > > > >> >>>>>>>>> Hi devs, >> > > > > > > > > > >> >>>>>>>>> >> > > > > > > > > > >> >>>>>>>>> I'd like to start a discussion about >> FLIP-436: >> > > > > Introduce >> > > > > > > > > "SHOW >> > > > > > > > > > >> >>>>>>>>> CREATE >> > > > > > > > > > >> >>>>>>>>> CATALOG" Syntax [1]. >> > > > > > > > > > >> >>>>>>>>> >> > > > > > > > > > >> >>>>>>>>> At present, the `SHOW CREATE TABLE` >> statement >> > > > > provides >> > > > > > > > > strong >> > > > > > > > > > >> >>>>>>>>> support >> > > > > > > > > > >> >>>>>>>>> for >> > > > > > > > > > >> >>>>>>>>> users to easily >> > > > > > > > > > >> >>>>>>>>> reuse created tables. However, despite the >> > > > > increasing >> > > > > > > > > > importance >> > > > > > > > > > >> >>>>>>>>> of the >> > > > > > > > > > >> >>>>>>>>> `Catalog` in user's >> > > > > > > > > > >> >>>>>>>>> business, there is no similar statement for >> > > users to >> > > > > > > use. >> > > > > > > > > > >> >>>>>>>>> >> > > > > > > > > > >> >>>>>>>>> According to the online discussion in >> > > FLINK-24939 >> > > > > [2] >> > > > > > > with >> > > > > > > > > > Jark >> > > > > > > > > > >> >>>>>>>>> Wu >> > > > > > > > > > >> >>>>>>>>> and >> > > > > > > > > > >> >>>>>>>>> Feng >> > > > > > > > > > >> >>>>>>>>> Jin, since `CatalogStore` >> > > > > > > > > > >> >>>>>>>>> has been introduced in FLIP-295 [3], we >> could >> > > use >> > > > > this >> > > > > > > > > > component >> > > > > > > > > > >> >>>>>>>>> to >> > > > > > > > > > >> >>>>>>>>> implement such a long-awaited >> > > > > > > > > > >> >>>>>>>>> feature, Please refer to the document [1] >> for >> > > > > > > > implementation >> > > > > > > > > > >> >>>>>>>>> details. >> > > > > > > > > > >> >>>>>>>>> >> > > > > > > > > > >> >>>>>>>>> examples as follows: >> > > > > > > > > > >> >>>>>>>>> >> > > > > > > > > > >> >>>>>>>>> Flink SQL> create catalog cat2 WITH >> > > > > > > > > > ('type'='generic_in_memory', >> > > > > > > > > > >> >>>>>>>>> >> > > > > > > > > > >> >>>>>>>>>> 'default-database'='db'); >> > > > > > > > > > >> >>>>>>>>>> [INFO] Execute statement succeeded. >> > > > > > > > > > >> >>>>>>>>>> Flink SQL> show create catalog cat2; >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> +----------------------------------------------------------------------------------------+ >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>>>>>>>> | result | >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> +----------------------------------------------------------------------------------------+ >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>>>>>>>> | CREATE CATALOG `cat2` WITH ( >> > > > > > > > > > >> >>>>>>>>>> 'default-database' = 'db', >> > > > > > > > > > >> >>>>>>>>>> 'type' = 'generic_in_memory' >> > > > > > > > > > >> >>>>>>>>>> ) >> > > > > > > > > > >> >>>>>>>>>> | >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> +----------------------------------------------------------------------------------------+ >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>>>>>>>> 1 row in set >> > > > > > > > > > >> >>>>>>>>> >> > > > > > > > > > >> >>>>>>>>> Looking forward to hearing from you, >> thanks! >> > > > > > > > > > >> >>>>>>>>> >> > > > > > > > > > >> >>>>>>>>> Best regards, >> > > > > > > > > > >> >>>>>>>>> Yubin >> > > > > > > > > > >> >>>>>>>>> >> > > > > > > > > > >> >>>>>>>>> [1] >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=296290756 >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>>>>>>> [2] >> > > > > https://issues.apache.org/jira/browse/FLINK-24939 >> > > > > > > > > > >> >>>>>>>>> [3] >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >>>> >> > > > > > > > > > >> >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> https://cwiki.apache.org/confluence/display/FLINK/FLIP-295%3A+Support+lazy+initialization+of+catalogs+and+persistence+of+catalog+configurations >> > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > >> > > >> >