Skip to content

Conversation

killme2008
Copy link
Contributor

@killme2008 killme2008 commented Sep 25, 2025

What's Changed in this PR

Describe the change in this PR

  • Fixed broken anchor links
  • Clarified that last_non_null mode keeps the latest non-null values rather than just latest values
  • Added documentation for modifying TTL policies using ALTER statements

Checklist

  • Please confirm that all corresponding versions of the documents have been revised.
  • Please ensure that the content in sidebars.ts matches the current document structure when you changed the document structure.
  • This change requires follow-up update in localized docs.

Signed-off-by: Dennis Zhuang <[email protected]>
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the data management documentation by fixing anchor links, clarifying merge mode behavior, and adding information about TTL management. The changes enhance user understanding of data operations in GreptimeDB.

  • Fixed broken anchor links with double hash symbols
  • Clarified that last_non_null mode keeps the latest non-null values rather than just latest values
  • Added documentation for modifying TTL policies using ALTER statements
  • Added important notes about TTL expiration timing and compaction behavior

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
versioned_docs/version-0.17/user-guide/manage-data/overview.md Fixed anchor link, added TTL expiration note and ALTER examples
versioned_docs/version-0.17/reference/sql/create.md Clarified last_non_null merge mode behavior
i18n/zh/docusaurus-plugin-content-docs/version-0.17/user-guide/manage-data/overview.md Chinese translation updates matching English changes
i18n/zh/docusaurus-plugin-content-docs/version-0.17/reference/sql/create.md Chinese translation for merge mode clarification
i18n/zh/docusaurus-plugin-content-docs/current/user-guide/manage-data/overview.md Current version Chinese translation updates
i18n/zh/docusaurus-plugin-content-docs/current/reference/sql/create.md Current version Chinese merge mode clarification
docs/user-guide/manage-data/overview.md Main docs with fixed anchor link and TTL ALTER examples
docs/reference/sql/create.md Main docs clarification of merge mode behavior

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Signed-off-by: Dennis Zhuang <[email protected]>
Copy link

cloudflare-workers-and-pages bot commented Sep 25, 2025

Deploying greptime-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0e3f8d3
Status: ✅  Deploy successful!
Preview URL: https://9ea837bb.greptime-docs.pages.dev
Branch Preview URL: https://feature-tweak-manage-data.greptime-docs.pages.dev

View logs

Copy link
Contributor

@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 merged commit 23f56ed into main Sep 25, 2025
6 checks passed
@fengjiachun fengjiachun deleted the feature/tweak-manage-data branch September 25, 2025 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants