Skip to content

Skip rendering of the table alias in SQL if not needed #1757

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

Merged
merged 4 commits into from
May 28, 2025

Conversation

plypaul
Copy link
Contributor

@plypaul plypaul commented May 16, 2025

This PR changes SQL generation to not render the table alias if not needed. This results in slightly cleaner SQL.

@cla-bot cla-bot bot added the cla:yes label May 16, 2025
@plypaul plypaul marked this pull request as ready for review May 16, 2025 16:59
@plypaul plypaul requested a review from a team as a code owner May 16, 2025 16:59
@plypaul plypaul force-pushed the p/semantic_graph__24 branch from 25dc597 to 4c9590f Compare May 16, 2025 17:17
@plypaul plypaul force-pushed the p/semantic_graph__25 branch from db284a6 to 0818790 Compare May 16, 2025 17:17
@plypaul plypaul force-pushed the p/semantic_graph__24 branch from 4c9590f to f22bef2 Compare May 27, 2025 23:24
@plypaul plypaul force-pushed the p/semantic_graph__25 branch from 0818790 to d5d1cf2 Compare May 27, 2025 23:24
@plypaul plypaul force-pushed the p/semantic_graph__24 branch from f22bef2 to 89f2c6f Compare May 27, 2025 23:28
@plypaul plypaul force-pushed the p/semantic_graph__25 branch 2 times, most recently from b0056ae to 0498755 Compare May 27, 2025 23:37
@plypaul plypaul force-pushed the p/semantic_graph__24 branch from 0dfb870 to aaee947 Compare May 27, 2025 23:38
@plypaul plypaul force-pushed the p/semantic_graph__25 branch from 0498755 to e070fad Compare May 27, 2025 23:38
Base automatically changed from p/semantic_graph__24 to main May 27, 2025 23:46
@plypaul plypaul force-pushed the p/semantic_graph__25 branch from e070fad to a6191a3 Compare May 27, 2025 23:46
@plypaul plypaul merged commit bd59066 into main May 28, 2025
12 checks passed
@plypaul plypaul deleted the p/semantic_graph__25 branch May 28, 2025 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants