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 > > > > >
