Skip to content

Conversation

@fengys1996
Copy link
Contributor

@fengys1996 fengys1996 commented Dec 29, 2025

What's Changed in this PR

This pr mainly add a description of the trigger status to enhance the trigger doc.

TODO:

  • sync change to version 1

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.

@killme2008
Copy link
Contributor

Is it ready for review? @fengys1996

@fengys1996
Copy link
Contributor Author

Is it ready for review? @fengys1996

It's still a draft, and there are still some things to do.

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 enhances the trigger documentation by adding comprehensive coverage of trigger status and state management. The update transforms the documentation from a Vector-based monitoring example to a clearer, self-contained tutorial focused on alert state transitions.

Key Changes:

  • Added a "Key Features" section highlighting SQL-native design, multi-stage state management, rich context, and ecosystem compatibility
  • Restructured the quick start example to use a simpler load1 table with a bash script for data simulation instead of Vector
  • Added detailed coverage of trigger status queries and alert state transitions (pending → firing → inactive) with example outputs
  • Expanded the reference section to include links to INFORMATION_SCHEMA.TRIGGERS and INFORMATION_SCHEMA.ALERTS

Reviewed changes

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

File Description
docs/enterprise/trigger.md English version enhanced with state management details, simplified examples, and new status query sections
i18n/zh/docusaurus-plugin-content-docs/current/enterprise/trigger.md Chinese translation with equivalent content updates

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

@fengys1996 fengys1996 marked this pull request as ready for review December 31, 2025 08:32
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.

2 participants