paid-media: add optional denormalized-names fieldgroup for summary metrics#2186
Closed
ejsuncy wants to merge 2 commits into
Closed
paid-media: add optional denormalized-names fieldgroup for summary metrics#2186ejsuncy wants to merge 2 commits into
ejsuncy wants to merge 2 commits into
Conversation
…trics Adds core-paid-media-denormalized-names fieldgroup carrying inline entity display names (account, campaign, ad group, ad, experience, asset, portfolio) on the paid media summary metrics dataset. Intended for ingestion shapes where lookup datasets are not available (e.g., MCA single-dataset feeds); lookup-driven ingestions leave the block null and continue resolving names via the corresponding lookup datasets. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Adds example.8 illustrating the MCA single-dataset use case: a summary row with denormalizedNames.* populated inline (no lookup datasets required). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Contributor
Author
|
Replacing with a properly-tagged PR linked to #2187. Reopening from a renamed branch following CONTRIBUTING.md guidance. |
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
1 participant
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
Adds a new optional fieldgroup
core-paid-media-denormalized-namesand references it from thepaid-media-summary-metricsschema.The fieldgroup carries inline entity display names (
accountName,campaignName,adGroupName,adName,experienceName,assetName,portfolioName) on each summary row.Motivation
The current Paid Media model expects entity names to be resolved via the per-entity lookup datasets (
paid-media-campaign-lookup,paid-media-ad-group-lookup, etc.) to avoid duplicating name strings on every summary row.For the MCA (Marketing Campaign Analytics) single-dataset use case — where a customer brings their own paid media data in a single flat feed without separate lookup datasets — there is no way today to surface entity names in reporting.
This change introduces an optional fieldgroup so:
xdm:paidMedia.xdm:denormalizedNames.*null and resolve names via lookups (no regression, no bloat).Files
components/fieldgroups/paid-media/core-paid-media-denormalized-names.schema.jsoncomponents/fieldgroups/paid-media/core-paid-media-denormalized-names.example.1.jsonschemas/paid-media/paid-media-summary-metrics.schema.json— added$refto the new fieldgroup inallOf.Notes
meta:status: experimentalconsistent with the rest of the paid-media schemas.