WIP: Delivery dashboard on delivery table with weight#157
Draft
davidnuescheler wants to merge 5 commits intomainfrom
Draft
WIP: Delivery dashboard on delivery table with weight#157davidnuescheler wants to merge 5 commits intomainfrom
delivery table with weight#157davidnuescheler wants to merge 5 commits intomainfrom
Conversation
- Point delivery dashboard at helix_logs_production.delivery - Multiply counts via sum(weight) / sumIf(weight, ...) across breakdowns, time series, anomaly investigation, and facet search - Add time-series-delivery.sql; disable SAMPLE and sample_hash dedup for pre-weighted rows - Centralize aggregation helpers in query-aggregations.js Made-with: Cursor
Preview deploymentPreview is live at: https://klickhaus.aemstatus.net/preview/pr-157/dashboard.html Updated for commit 15c2ed2 |
- Add "Last 3 days" (3d) time range between 24h and 7d - Replace client IP facet with cdn.originating_ip as "Originating IPs" - Remove Cache Status, Accept, and Cache-Control (Req) facets - Add Content Encoding and Surrogate Keys facets (delivery table) - Hide Redirect Location, Content Encoding, and Surrogate Keys by default Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ix facet routing - Add backend.html dashboard page for per-subsystem backend traffic analysis - Add backend-main.js and time-series-backend.sql for weighted query support - Fix canUseFacetTable() to skip cdn_facet_minutes for non-cdn_requests_v2 tables - Fix bucketed facet filters (Content Length, Response Time) by adding evaluated multiIf expressions to the filter column allowlist - Fix Content Encoding facet to show (empty) entry for unencoded responses - Add RSO facet for helix.rso backend table column - Link backend dashboard from main index page Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Work in progress: base
delivery.htmlon the ClickHousedeliverytable instead ofcdn_requests_v2, and scale all displayed counts using theweightcolumn (sum(weight),sumIf(weight, …)). Includes time series, breakdowns, anomaly investigation queries, facet search, and shared helpers injs/query-aggregations.js. Sampling is disabled for this dashboard so weights are not double-scaled.Testing Done
npm test/npm run lintpending).Checklist
npm test)npm run lint)Made with Cursor