Skip to content

Conversation

@fengjiachun
Copy link
Contributor

@fengjiachun fengjiachun commented Oct 31, 2025

What's Changed in this PR

As the title

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.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 31, 2025

Deploying greptime-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 78d80ec
Status: ✅  Deploy successful!
Preview URL: https://2b897d3f.greptime-docs.pages.dev
Branch Preview URL: https://chore-mimor-changes.greptime-docs.pages.dev

View logs

@killme2008 killme2008 requested a review from Copilot November 4, 2025 06:54
Copy link
Contributor

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 updates the GreptimeDB documentation to reflect new configuration options and make minor corrections. The changes primarily focus on expanding configuration documentation with newly added options for various components and protocols.

  • Adds comprehensive configuration options for HTTP, gRPC, MySQL, PostgreSQL, Jaeger, and storage components
  • Updates logging, caching, and index configuration documentation
  • Corrects architectural details and improves text clarity

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
i18n/zh/docusaurus-plugin-content-docs/current/user-guide/deployments-administration/configuration.md Adds extensive new configuration options and updates existing documentation tables with detailed descriptions for protocol options, storage, caching, logging, and region engine settings
i18n/zh/docusaurus-plugin-content-docs/current/user-guide/concepts/why-greptimedb.md Updates Chinese pronouns for consistency and fixes spacing
i18n/zh/docusaurus-plugin-content-docs/current/user-guide/concepts/overview.md Corrects verb usage in architecture description
i18n/zh/docusaurus-plugin-content-docs/current/user-guide/concepts/data-model.md Updates terminology from "expiration management" to "lifecycle management"
i18n/zh/docusaurus-plugin-content-docs/current/user-guide/concepts/architecture.md Corrects minimum node requirement from three to two
docs/user-guide/deployments-administration/configuration.md Mirrors Chinese version changes with extensive new configuration options and updated documentation tables
docs/user-guide/concepts/architecture.md Corrects minimum node requirement from three to two

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@killme2008 killme2008 left a comment

Choose a reason for hiding this comment

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

The document not only lists sample configurations but also explains key ones. It serves as a user guide, not a reference.

@fengjiachun fengjiachun marked this pull request as draft November 4, 2025 07:42
Signed-off-by: jeremyhi <fengjiachun@gmail.com>
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