Skip to content

Conversation

zhongzc
Copy link
Contributor

@zhongzc zhongzc commented Sep 7, 2025

I hereby agree to the terms of the GreptimeDB CLA.

Refer to a related PR or issue link (optional)

Depends on #6921

What's changed and what's your intention?

Introduces information_schema.ssts_*, a system table exposing SST file metadata to aid debugging and observability.

SELECT * FROM information_schema.ssts_manifest;
SELECT * FROM information_schema.ssts_storage;

PR Checklist

Please convert it to a draft if some of the following conditions are not met.

  • I have written the necessary rustdoc comments.
  • I have added the necessary unit tests and integration tests.
  • This PR requires documentation updates.
  • API changes are backward compatible.
  • Schema or data changes are backward compatible.

@zhongzc zhongzc requested a review from a team as a code owner September 7, 2025 10:59
@github-actions github-actions bot added size/M docs-required This change requires docs update. labels Sep 7, 2025
@github-actions github-actions bot added docs-not-required This change does not impact docs. and removed docs-required This change requires docs update. labels Sep 7, 2025
@killme2008
Copy link
Contributor

It seems this information is for diagnostics or internal use. I’m not sure whether it belongs in the information schema or the greptime private schema.

@fengjiachun @sunng87 @evenyag WDYT

Base automatically changed from zhongzc/wire-dn-fe-inspect to main September 9, 2025 03:59
@github-actions github-actions bot added docs-required This change requires docs update. and removed docs-not-required This change does not impact docs. labels Sep 9, 2025
@zhongzc zhongzc force-pushed the zhongzc/ssts-systable branch from 6f73787 to c7b5bb2 Compare September 9, 2025 08:26
Copy link
Contributor

@evenyag evenyag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. We can put them under the information schema, but only for debugging purposes, and don't provide guarantees about compatibility. We can mention this in the document.

@fengjiachun
Copy link
Collaborator

LGTM. We can put them under the information schema, but only for debugging purposes, and don't provide guarantees about compatibility. We can mention this in the document.

+1

Copy link
Collaborator

@fengjiachun fengjiachun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fengjiachun fengjiachun added this pull request to the merge queue Sep 16, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 16, 2025
@evenyag
Copy link
Contributor

evenyag commented Sep 16, 2025

@zhongzc The sqlness test is unstable.

@killme2008
Copy link
Contributor

Failed by status checker while merging. @zhongzc

@killme2008 killme2008 added this pull request to the merge queue Sep 23, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 23, 2025
Signed-off-by: Zhenchi <[email protected]>
@zhongzc zhongzc enabled auto-merge September 23, 2025 10:15
Signed-off-by: Zhenchi <[email protected]>
@zhongzc zhongzc added this pull request to the merge queue Sep 23, 2025
Merged via the queue into main with commit 80c8ab4 Sep 23, 2025
42 checks passed
@zhongzc zhongzc deleted the zhongzc/ssts-systable branch September 23, 2025 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-required This change requires docs update. size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants