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

Add limited GROUP BY support #43

Merged
merged 8 commits into from
Mar 17, 2025
Merged

Add limited GROUP BY support #43

merged 8 commits into from
Mar 17, 2025

Conversation

levkk
Copy link
Collaborator

@levkk levkk commented Mar 17, 2025

Feature

  • Add GROUP BY support to aggregates. This works currently without rewriting queries, so the GROUP BY clause must use references to columns present in the result, e.g. GROUP BY 3, 4.
  • Support NULLs in DataRow message building.

#40

levkk added 2 commits March 17, 2025 14:07
@levkk levkk changed the title Add GROUP BY support Add limited GROUP BY support Mar 17, 2025
@levkk levkk marked this pull request as ready for review March 17, 2025 21:09
levkk added 6 commits March 17, 2025 14:11
@levkk levkk merged commit 3d1498b into main Mar 17, 2025
3 checks passed
@levkk levkk deleted the levkk-group-by branch March 17, 2025 21:49
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.

None yet

1 participant