davidradl commented on code in PR #26393: URL: https://github.com/apache/flink/pull/26393#discussion_r2026653613
########## docs/content/docs/libs/state_processor_api.md: ########## @@ -515,6 +515,28 @@ Before you interrogate state using the table API, make sure to review our [Flink IMPORTANT NOTE: State Table API only supports keyed state. +### Metadata + +The SQL connector allows users to read metadata of savepoints in the following way: Review Comment: nit: read metadata -> read the metadata This is not a SQL connector - it is a new table function. ########## docs/content/docs/libs/state_processor_api.md: ########## @@ -515,6 +515,28 @@ Before you interrogate state using the table API, make sure to review our [Flink IMPORTANT NOTE: State Table API only supports keyed state. +### Metadata + +The SQL connector allows users to read metadata of savepoints in the following way: +```SQL +LOAD MODULE state; Review Comment: nit: if this is savepoints only ; should we call it savepoint-state. Currently the implementation seems to be tied to a fixed savepoint schema. savepoint-state would be more descriptive and better scoped - allowing us to provide a more generic state module later - if we want to. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
