Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
Hi. Yubin. Thanks for your update. LGTM. Best, Hang Yubin Li 于2024年3月20日周三 11:56写道: > Hi Hang, > > I have updated FLIP as you suggested, thanks for your valuable feedback! > > Best, > Yubin > > On Wed, Mar 20, 2024 at 11:15 AM Hang Ruan wrote: > > > > Hi, Yubin, > > > > I found a little mistake in FLIP. > > `ALTER CATALOG catalog_name RESET (key1=val1, key2=val2, ...)` should be > > changed as `ALTER CATALOG catalog_name RESET (key1, key2, ...)`, right? > > > > Best, > > Hang > > > > > > Lincoln Lee 于2024年3月20日周三 10:04写道: > > > > > 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 于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 于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 > > > > >> 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 于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 于2024年3月18日周一 17:44写道: > > > >> > > > > > > >> > > > > Hi Jark, > > > >> > > > > > > > >> > > > > Thanks for your response, I have updated FLIP-436:
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
Hi Hang, I have updated FLIP as you suggested, thanks for your valuable feedback! Best, Yubin On Wed, Mar 20, 2024 at 11:15 AM Hang Ruan wrote: > > Hi, Yubin, > > I found a little mistake in FLIP. > `ALTER CATALOG catalog_name RESET (key1=val1, key2=val2, ...)` should be > changed as `ALTER CATALOG catalog_name RESET (key1, key2, ...)`, right? > > Best, > Hang > > > Lincoln Lee 于2024年3月20日周三 10:04写道: > > > 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 于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 于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 > > >> 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 于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 > > > > >> > > 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 于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 > > >>
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
Hi, Yubin, I found a little mistake in FLIP. `ALTER CATALOG catalog_name RESET (key1=val1, key2=val2, ...)` should be changed as `ALTER CATALOG catalog_name RESET (key1, key2, ...)`, right? Best, Hang Lincoln Lee 于2024年3月20日周三 10:04写道: > 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 于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 于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 > >> 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 于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 > > >> > > 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 于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 > 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 > >> wrote: > >> > > > > > > >> > > > > > > Hi Jark, > >> > > > > > > > >> > > > > > > Good pointing! Thanks for your
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 于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 于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 >> 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 于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 >> > > 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 于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 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 >> 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 >>
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 于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 > 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 于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 > > > 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 于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 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 > 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 |
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 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 于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 > > 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 于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 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 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)) > > > > > > | > > > > > > > > > > > > > > > > > >
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 于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 > 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 于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 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 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 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, > > > > > >
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 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 于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 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 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 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 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 :)
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 于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 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 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 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 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 > 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
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 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 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 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 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 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" wrote: > > > > > >Hi folks, > > > > > > > > > > > >Thank you all for your input, it really makes sense to introduce > > > missing > > > > > >catalog-related SQL
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 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 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 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 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" 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 >
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 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 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 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" 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 > > > > > > > >
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 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 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" 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 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 写道: > > >> > > > >> > Hi, Yubin > > >> > > > >> > > > >> > Thank you for initiating this discussion! +1 for the proposal. > > >> > > > >> > > > >> > > > >> > > > >> > > > >> > > > >> > Best, > > >> > Yuepeng Pan > > >> > > > >> > > > >> > > > >> > > > >> > > > >> > > > >> > > > >> > > > >> > > > >> > At 2024-03-14 18:57:35, "Ferenc Csaky" > > >> 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
Re: Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 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" 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 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 写道: > >> > > >> > Hi, Yubin > >> > > >> > > >> > Thank you for initiating this discussion! +1 for the proposal. > >> > > >> > > >> > > >> > > >> > > >> > > >> > Best, > >> > Yuepeng Pan > >> > > >> > > >> > > >> > > >> > > >> > > >> > > >> > > >> > > >> > At 2024-03-14 18:57:35, "Ferenc Csaky" > >> 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] > >> > >> > >> >
Re:Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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" 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 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 写道: >> > >> > Hi, Yubin >> > >> > >> > Thank you for initiating this discussion! +1 for the proposal. >> > >> > >> > >> > >> > >> > >> > Best, >> > Yuepeng Pan >> > >> > >> > >> > >> > >> > >> > >> > >> > >> > At 2024-03-14 18:57:35, "Ferenc Csaky" >> 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] >>
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 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 写道: > > > > Hi, Yubin > > > > > > Thank you for initiating this discussion! +1 for the proposal. > > > > > > > > > > > > > > Best, > > Yuepeng Pan > > > > > > > > > > > > > > > > > > > > At 2024-03-14 18:57:35, "Ferenc Csaky" > 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
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 写道: > > Hi, Yubin > > > Thank you for initiating this discussion! +1 for the proposal. > > > > > > > Best, > Yuepeng Pan > > > > > > > > > > At 2024-03-14 18:57:35, "Ferenc Csaky" 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 >> 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
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
Hi, Yubin Thank you for initiating this discussion! +1 for the proposal. Best, Yuepeng Pan At 2024-03-14 18:57:35, "Ferenc Csaky" 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 >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. >> > > > > > > >> > > > > > >
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
Hi, Yubin. Thanks for leading the discussion. +1 for the FLIP. Best, Zhongqiang Gong Ferenc Csaky 于2024年3月14日周四 18:58写道: > 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 > 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 > > >
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 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 > > > > > >
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
+1 for the FLIP, thanks Yubin for driving it. Also +1 to complete the whole story about Catalog in this FLIP as Jark and Jane said above. Jane Chan 于2024年3月14日周四 15:33写道: > > 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 wrote: > > > Hi, Yubin. > > > > Thanks for the FLIP. +1 for it. > > > > Best, > > Hang > > > > Yubin Li 于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 > > > 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 > > 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 > > > > > 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 > > 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'); > > >
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 wrote: > Hi, Yubin. > > Thanks for the FLIP. +1 for it. > > Best, > Hang > > Yubin Li 于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 > > 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 > 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 > > > > 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 > 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 ( > > > > > > >
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
Hi, Yubin. Thanks for the FLIP. +1 for it. Best, Hang Yubin Li 于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 > 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 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 > > > 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 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] > > > > > > > > > > > > >
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
Thank you Yubin, +1 for the proposal. We have been lacking catalog related syntax to operate catalogs. It's a good chance to complete the syntax as we have introduced CatalogStore. >From what I can see, some useful commands are still missing for catalogs, such as, alter catalog, describe catalog. What do you think about including these syntaxes in the FLIP as well? Best, Jark On Thu, 14 Mar 2024 at 10:16, Yubin Li wrote: > 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 > 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 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 > > > 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 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
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 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 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 > > 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 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 > > > > >
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 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 > 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 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 > > >
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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 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 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 >
Re: [DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
+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 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
[DISCUSS] FLIP-436: Introduce "SHOW CREATE CATALOG" Syntax
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