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

Implement remaining TSQL set operations. #1227

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Conversation

asnare
Copy link
Contributor

@asnare asnare commented Nov 21, 2024

Changes

This PR implements support for parsing set operations with TSql: UNION [ALL], EXCEPT and INTERSECT.

The grammar previously supported these but they were not being converted to the IR.

Linked issues

Resolves #1126.
Resolves #1102.

Tests

  • added unit tests
  • added transpiler tests
  • added functional tests

@asnare asnare added enhancement New feature or request feat/ir everything related to abstract syntax trees tech debt design flaws and other cascading effects sql/tsql labels Nov 21, 2024
@asnare asnare self-assigned this Nov 21, 2024
Copy link

github-actions bot commented Nov 21, 2024

Coverage tests results

459 tests  +3   422 ✅ +3   4s ⏱️ ±0s
  6 suites ±0    37 💤 ±0 
  6 files   ±0     0 ❌ ±0 

Results for commit 0f4cf7a. ± Comparison against base commit 9dcc986.

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feat/ir everything related to abstract syntax trees sql/tsql tech debt design flaws and other cascading effects
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TODO] Implement set ops [TODO] This will change when UNION is implemented correctly
2 participants