Skip to content

Add metric view YAML serde package - #6837

Draft
pietern wants to merge 12 commits into
mainfrom
metricviews-serde
Draft

pietern wants to merge 12 commits into
mainfrom
metricviews-serde

Conversation

@pietern

@pietern pietern commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Changes

  • Add libs/metricviews with typed YAML parsing and serialization for versions 0.1, 1.0, and 1.1, including single-source and multi-source views.
  • Normalize backend-equivalent values for drift comparisons, and cover the supported shapes with 47 generic fixtures.
  • Preserve quoted parameter defaults and YAML aliases; reject unsupported v1.1 view types and multi-source shapes without their discriminator.

Why

Provide reusable Go parsing and serialization for Databricks metric view YAML, independent of bundle resource wiring.

Tests

  • go test ./libs/metricviews -count=1
  • Full repository checks were skipped as requested.

PR description drafted with Codex.

@pietern pietern changed the title metricviews serde Add metric view YAML serde package Sep 24, 2026
@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: f02a1e7

Run: 36119946077

Env ✅​pass 🙈​skip Time
✅​ aws linux 276 15 5:34
✅​ aws windows 278 13 3:25
✅​ azure linux 275 15 6:16
✅​ azure windows 277 13 6:41
✅​ gcp linux 276 15 5:23
✅​ gcp windows 278 13 3:32
Top 6 slowest tests (at least 2 minutes):
duration env testname
6:38 azure windows TestAccept
4:11 aws linux TestAccept
4:02 gcp linux TestAccept
3:53 azure linux TestAccept
3:31 gcp windows TestAccept
3:23 aws windows TestAccept

This branch has not been deployed

No deployments
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