console-1968-improve-most-valuable-alert-types#7935
console-1968-improve-most-valuable-alert-types#7935jonathanawesome wants to merge 24 commits intomainfrom
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request outlines a proposal to enhance the alerts and notifications system by adding email support and introducing metric-based alerts for latency, error rates, and traffic. The review identifies several critical technical considerations for the implementation: the inability to run certain PostgreSQL type alterations within transactions, potential inaccuracies in ClickHouse metric calculations due to interval snapping, the need for zero-division handling in percentage change formulas, and a consistency error regarding the proposed evaluation frequency.
|
🐋 This PR was built and pushed to the following Docker images: Targets: Platforms: Image Tag: |
🚀 Snapshot Release (
|
| Package | Version | Info |
|---|---|---|
@graphql-hive/laboratory |
0.1.4-alpha-20260417152317-35cc93a53b27687145e8bb88394b0399f1cc982f |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/render-laboratory |
0.1.4-alpha-20260417152317-35cc93a53b27687145e8bb88394b0399f1cc982f |
npm ↗︎ unpkg ↗︎ |
hive |
11.0.3-alpha-20260417152317-35cc93a53b27687145e8bb88394b0399f1cc982f |
npm ↗︎ unpkg ↗︎ |
1 participant
This PR will eventually contain the implementation for our enhanced alerting features. For now, a draft PR with the Claude-generated backend plan as an RFC prior to implementation and while design work is ongoing.