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.

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

Warning on column dupliactes #24

Open
mmaelicke opened this issue Jan 15, 2021 · 0 comments
Open

Warning on column dupliactes #24

mmaelicke opened this issue Jan 15, 2021 · 0 comments
Labels
bug Something isn't working exercise-js Issue relates to the exercise application legacy This issue affects the legacy exercises appplication and might not be solved
Milestone
Version 1.0.0

Comments

@mmaelicke
Copy link
Member

With SQL it's possible to query the same Column-alias within a single SQL query. Then, the exercises app will overwrite one columns with the other, as they share the same key in the SqlResult data model. This can be really bad, as the actual SQL result returned by the database differs and will cause CREATE statements based on these selects to fail without any obvious reason for the user.

Either implement a warning, if there are duplicates, or suffix the columns like _1 etc. Or both.
@mmaelicke mmaelicke added bug Something isn't working exercise-js Issue relates to the exercise application labels Jan 15, 2021
@mmaelicke mmaelicke added this to the Version 1.0.0 milestone Jan 15, 2021
@mmaelicke mmaelicke added the legacy This issue affects the legacy exercises appplication and might not be solved label Dec 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working exercise-js Issue relates to the exercise application legacy This issue affects the legacy exercises appplication and might not be solved
Projects
None yet
Milestone
Version 1.0.0
Development

No branches or pull requests
1 participant
@mmaelicke