docs(spec): a count tile over an aggregating report sums its count measure - #50
Open
delchev wants to merge 1 commit into
Open
docs(spec): a count tile over an aggregating report sums its count measure#50delchev wants to merge 1 commit into
delchev wants to merge 1 commit into
Conversation
…asure A report declaring measures yields one row per group, so counting its rows shows the number of groups - four statuses in place of fourteen requests. Say what the count is, that an aggregating report must declare `count(*)`, and that one without it is rejected rather than tiled with the group count. Co-Authored-By: Claude Opus 5 <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
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.
Pairs with IntentFile/intent-specification#67.
kind: countread as "the number of records the report yields" left a report declaringmeasures:ambiguous: its rows are groups, so counting them shows four statuses where the records number fourteen. The page now states the count as thecount(*)measure summed over the rows, requires that measure on an aggregating report (rejected at generation without it), and keeps the row count for a report that does not aggregate.Upstream fix: eclipse-dirigible/dirigible#7113 (issue eclipse-dirigible/dirigible#7102).
🤖 Generated with Claude Code