Skip to content

docs: batching mode for Prometheus Remote Write protocol#2392

Open
v0y4g3r wants to merge 3 commits intomainfrom
docs/prom-batching
Open

docs: batching mode for Prometheus Remote Write protocol#2392
v0y4g3r wants to merge 3 commits intomainfrom
docs/prom-batching

Conversation

@v0y4g3r
Copy link
Copy Markdown
Contributor

@v0y4g3r v0y4g3r commented Apr 7, 2026

What's Changed in this PR

closes #2375

Add batching mode for Prometheus Remote Write

  • Configuration Updates: Enhanced [prom_store] section in configuration.md to include new batching options such as pending_rows_flush_interval, max_batch_rows, max_concurrent_flushes, worker_channel_capacity, and max_inflight_requests.
  • Documentation Enhancements: Added detailed descriptions for new batching mode options in prometheus.md to improve ingestion throughput.
  • Localization: Updated Chinese documentation in configuration.md and prometheus.md to reflect new batching mode features and options.

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.

 ### Add batching mode for Prometheus Remote Write

 - **Configuration Updates**: Enhanced `[prom_store]` section in `configuration.md` to include new batching options such as `pending_rows_flush_interval`, `max_batch_rows`, `max_concurrent_flushes`, `worker_channel_capacity`, and
 `max_inflight_requests`.
 - **Documentation Enhancements**: Added detailed descriptions for new batching mode options in `prometheus.md` to improve ingestion throughput.
 - **Localization**: Updated Chinese documentation in `configuration.md` and `prometheus.md` to reflect new batching mode features and options.

Signed-off-by: Lei, HUANG <mrsatangel@gmail.com>
Copilot AI review requested due to automatic review settings April 7, 2026 07:51
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 7, 2026

Deploying greptime-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: b0ed3d2
Status: ✅  Deploy successful!
Preview URL: https://efed87ad.greptime-docs.pages.dev
Branch Preview URL: https://docs-prom-batching.greptime-docs.pages.dev

View logs

Copy link
Copy Markdown
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

Updates GreptimeDB documentation to describe the new Prometheus Remote Write batching mode and its configuration, including localized (zh) docs, to address #2375.

Changes:

  • Added a “Batching mode” section to Prometheus Remote Write performance tuning docs (EN + ZH), describing behavior and listing new tuning knobs.
  • Extended [prom_store] configuration examples and option tables (EN + ZH) with batching-related parameters and defaults.
  • Added cross-links from configuration docs to the new batching-mode documentation sections.

Reviewed changes

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

File Description
docs/user-guide/ingest-data/for-observability/prometheus.md Documents batching mode behavior and batching-related [prom_store] options (EN).
docs/user-guide/deployments-administration/configuration.md Adds batching-related [prom_store] keys to the sample config and option reference table (EN).
i18n/zh/docusaurus-plugin-content-docs/current/user-guide/ingest-data/for-observability/prometheus.md Chinese localization of batching mode section and option table (ZH).
i18n/zh/docusaurus-plugin-content-docs/current/user-guide/deployments-administration/configuration.md Chinese localization of [prom_store] batching config keys and reference table (ZH).

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

Comment thread docs/user-guide/ingest-data/for-observability/prometheus.md Outdated
Comment thread docs/user-guide/deployments-administration/configuration.md Outdated
Comment thread docs/user-guide/ingest-data/for-observability/prometheus.md Outdated
Signed-off-by: Lei, HUANG <mrsatangel@gmail.com>
Comment thread docs/user-guide/ingest-data/for-observability/prometheus.md Outdated
@fengjiachun
Copy link
Copy Markdown
Contributor

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4ac1cb04ef

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/user-guide/ingest-data/for-observability/prometheus.md Outdated
Copy link
Copy Markdown
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.

Others LGTM

 Remove duplicate entry in Prometheus documentation

 - Removed a duplicate description for `max_inflight_requests` in `prometheus.md`.

Signed-off-by: Lei, HUANG <mrsatangel@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.

Update docs for feat: pending rows batching for metrics

4 participants