clintropolis commented on code in PR #19830: URL: https://github.com/apache/druid/pull/19830#discussion_r3936802384
########## docs/development/extensions-core/catalog.md: ########## @@ -43,6 +43,177 @@ allowing queries to be more concise, and simpler to write. This also allows the written into a defined column of the table is consistent with that columns definition, minimizing errors where unexpected data is written into a particular column of the table. +### SQL DDL + +Tables can be defined with SQL instead of by posting a table specification. `CREATE TABLE` and `ALTER TABLE` are +submitted to the Broker like any other SQL statement, and write the same catalog metadata the REST API does. They +return no rows. + +These statements change catalog metadata only. They never create, modify, or delete segments: defining a table does Review Comment: catalog based compaction doesn't currently handle baseTable spec, but once in place it would apply all of the other things like projections and clustering and stuff. I'm going to omit catalog stuff from the docs for now and address this stuff in a follow-up branch I have in progress that wires that up so that it is consistent with the rest of the catalog and (also make catalog compaction config work with MSQ compaction since that didn't work either). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
