gaborgsomogyi commented on code in PR #26393:
URL: https://github.com/apache/flink/pull/26393#discussion_r2026962151


##########
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;
+SELECT * FROM savepoint_metadata('/root/dir/of/checkpoint-data/chk-1');

Review Comment:
   > The flip talks of checkpoints and savepoint state, can we use this 
funciton to access checkpoint state as well? 
   
   Yes. We can't name things in both ways. Initially we've agreed to call the 
whole state processor API things as `savepoint`... and the only exception here 
is the module because it's a reserved keyword in Flink SQL. If we go on this 
road, then the whole state processor API including internals must be renamed 
which I don't support.



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

Reply via email to