Skip to content

docs: enhance influxdb migration guide#2624

Open
MichaelScofield wants to merge 7 commits into
mainfrom
docs/enhance-influxdb-migration-guide
Open

docs: enhance influxdb migration guide#2624
MichaelScofield wants to merge 7 commits into
mainfrom
docs/enhance-influxdb-migration-guide

Conversation

@MichaelScofield

@MichaelScofield MichaelScofield commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

What's Changed in this PR

... from the blog

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

cloudflare-workers-and-pages Bot commented Jul 14, 2026

Copy link
Copy Markdown

Deploying greptime-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9d70bfe
Status: ✅  Deploy successful!
Preview URL: https://af332c98.greptime-docs.pages.dev
Branch Preview URL: https://docs-enhance-influxdb-migrat.greptime-docs.pages.dev

View logs

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 enhances the InfluxDB → GreptimeDB migration documentation (EN + zh) by correcting/clarifying write API examples (including timestamp precision) and expanding the shared migration guide content (data-model mapping, query migration hints, and validation steps).

Changes:

  • Update InfluxDB line protocol HTTP write examples to include precision=ns and use bucket for v2 write requests.
  • Expand the shared migration guide with data-model mapping tables, InfluxQL→SQL mapping guidance, and post-import validation steps.
  • Improve import shell snippets by quoting ${file} in --data-binary.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
docs/user-guide/migrate-to-greptimedb/migrate-from-influxdb.md Updates self-hosted migration write examples (v2 bucket + precision=ns; v1 precision=ns; safer --data-binary quoting).
docs/greptimecloud/migrate-to-greptimecloud/migrate-from-influxdb.md Updates GreptimeCloud migration write examples similarly (adds precision=ns; safer --data-binary quoting).
docs/db-cloud-shared/migrate/migrate-from-influxdb.md Adds richer shared migration guidance (data-model mapping, query migration mapping, validation steps).
i18n/zh/docusaurus-plugin-content-docs/current/user-guide/migrate-to-greptimedb/migrate-from-influxdb.md Mirrors self-hosted write-example updates in zh localized docs.
i18n/zh/docusaurus-plugin-content-docs/current/greptimecloud/migrate-to-greptimecloud/migrate-from-influxdb.md Mirrors GreptimeCloud write-example updates in zh localized docs.
i18n/zh/docusaurus-plugin-content-docs/current/db-cloud-shared/migrate/migrate-from-influxdb.md Mirrors shared migration-guide enhancements in zh localized docs.

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

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Comment thread docs/db-cloud-shared/migrate/migrate-from-influxdb.md Outdated
@MichaelScofield

Copy link
Copy Markdown
Contributor Author

@fengjiachun PTAL

Comment thread docs/db-cloud-shared/migrate/migrate-from-influxdb.md Outdated

```shell
curl -X POST 'http://<host>:4000/v1/influxdb/api/v2/write?db=<db-name>' \
curl -X POST 'http://<host>:4000/v1/influxdb/api/v2/write?bucket=<db-name>&precision=ns' \

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Since precision=ns is the default, is it necessary to include it in the URL?

@fengjiachun fengjiachun left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

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.

4 participants