Skip to content

improvement: support "unrelated" aggregates#2240

Merged
zachdaniel merged 11 commits intomainfrom
feature/unrelated-aggregates
Aug 11, 2025
Merged

improvement: support "unrelated" aggregates#2240
zachdaniel merged 11 commits intomainfrom
feature/unrelated-aggregates

Conversation

@zachdaniel
Copy link
Copy Markdown
Contributor

the terminology here is a bit confusing, but aggregates were originally designed over relationships. This allows to use a Resource, instead of a relationship.path in aggregates.

Contributor checklist

Leave anything that you believe does not apply unchecked.

  • I accept the AI Policy, or AI was not used in the creation of this PR.
  • Bug fixes include regression tests
  • Chores
  • Documentation changes
  • Features include unit/acceptance tests
  • Refactoring
  • Update dependencies

@zachdaniel
Copy link
Copy Markdown
Contributor Author

Will have to hold a bit on this because I'm realizing that the builtin exists expression needs support here for other things to work.

@zachdaniel zachdaniel marked this pull request as draft July 31, 2025 23:18
the terminology here is a bit confusing, but aggregates were originally
designed over relationships. This allows to use a Resource, instead of
a relationship.path in aggregates.
@zachdaniel zachdaniel force-pushed the feature/unrelated-aggregates branch from bc49d42 to 351e2f3 Compare August 11, 2025 01:30
@zachdaniel zachdaniel marked this pull request as ready for review August 11, 2025 03:34
@zachdaniel zachdaniel merged commit 471274d into main Aug 11, 2025
39 checks passed
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.

1 participant