Skip to content

docs: json2 data type - #2615

Merged
sunng87 merged 10 commits into
GreptimeTeam:mainfrom
fengys1996:feat/json2
Aug 3, 2026
Merged

docs: json2 data type#2615
sunng87 merged 10 commits into
GreptimeTeam:mainfrom
fengys1996:feat/json2

Conversation

@fengys1996

@fengys1996 fengys1996 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

What's Changed in this PR

This PR adds a guide to JSON2, including a quick start, syntax, and future roadmap.

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.

@fengys1996
fengys1996 requested a review from Copilot July 10, 2026 08:09
@fengys1996

Copy link
Copy Markdown
Contributor Author

@codex review

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 adds documentation for GreptimeDB’s JSON2 data type (nightly/current docs) and surfaces it in the Logs section of the docs sidebar, including a Chinese localized version.

Changes:

  • Added a new JSON2 documentation page under docs/user-guide/logs/.
  • Added the corresponding Chinese doc under i18n/zh/.../current/.
  • Linked the new page into the Logs sidebar navigation.

Reviewed changes

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

File Description
sidebars.ts Adds the new user-guide/logs/json2 doc entry to the Logs sidebar.
docs/user-guide/logs/json2.md Introduces the English JSON2 type guide with examples and limitations.
i18n/zh/docusaurus-plugin-content-docs/current/user-guide/logs/json2.md Adds the Chinese version of the JSON2 type guide.

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

Comment thread docs/user-guide/logs/json2.md Outdated
Comment thread i18n/zh/docusaurus-plugin-content-docs/current/user-guide/logs/json2.md Outdated

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

Copy link
Copy Markdown

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: eb656f1549

ℹ️ 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 sidebars.ts
@fengys1996
fengys1996 marked this pull request as ready for review July 10, 2026 08:24
Comment thread docs/user-guide/logs/json2.md Outdated
Comment thread docs/user-guide/logs/json2.md Outdated
Comment thread docs/user-guide/logs/json2.md Outdated
Comment thread docs/user-guide/logs/json2.md
Comment thread docs/user-guide/logs/json2.md Outdated
Comment thread docs/user-guide/logs/json2.md Outdated
Comment thread docs/user-guide/logs/json2.md Outdated
Comment thread docs/user-guide/logs/json2.md
Comment thread docs/user-guide/logs/json2.md Outdated
@fengys1996

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Bravo.

Reviewed commit: 74e21a5234

ℹ️ 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".

@sunng87
sunng87 merged commit 8b8d050 into GreptimeTeam:main Aug 3, 2026
6 checks passed
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.

5 participants