Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GroupBy JSON Entity #29011

Open
Tracked by #30173
smitpatel opened this issue Sep 7, 2022 · 0 comments
Open
Tracked by #30173

GroupBy JSON Entity #29011

smitpatel opened this issue Sep 7, 2022 · 0 comments

Comments

@smitpatel
Copy link
Member

Variation for #17653

When grouping by an entity, we add all the columns of the entity in the grouping key.
For entity which are mapped as JSON,

  • Should we add all the properties? If yes, how would we add the ordinal property which is non-persisted.
  • Given the JSON entity is going to be single SQL function call, we can just add the function call.

First option aligns with regular entity but would cause a lot of JSON scalar function calls. Second option will only have one call but the grouping key will contain owned entities in aggregate too. Further, grouping by a large JSON blob may not be something which user should do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants