Skip to content

Added logic to qualify stage names for Dedup and Group By operations #970

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.

Sign up for GitHub

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

Jump to bottom
Merged
fernst merged 1 commit into from
Apr 12, 2022
Merged

Added logic to qualify stage names for Dedup and Group By operations #970

fernst merged 1 commit into from
Apr 12, 2022

Conversation

@fernst
Copy link
Contributor

@fernst fernst commented Apr 12, 2022

Qualifying dataset names ensures stage names with spaces or SQL reserved words don't cause issues when building SQL statements

@fernst fernst added the build Trigger unit test build label Apr 12, 2022
@fernst fernst merged commit ee9a407 into develop Apr 12, 2022
@fernst fernst mentioned this pull request Apr 16, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tivv tivv tivv approved these changes

@sanjanasandeep sanjanasandeep sanjanasandeep approved these changes

@DJSagarAhire DJSagarAhire Awaiting requested review from DJSagarAhire

Assignees

No one assigned

Labels

bq-pushdown build Trigger unit test build

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@fernst @tivv @sanjanasandeep