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 ETL exercise #76

Merged
merged 4 commits into from
Jun 11, 2024
Merged

Add ETL exercise #76

merged 4 commits into from
Jun 11, 2024

Conversation

Steffan153
Copy link
Contributor

I think this one is pretty good for JSON handling.

@@ -0,0 +1,4 @@
"{""1"":[""A""]}",""
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the other maintainers, what are you thoughts on what is essentially a JSON exercise on a SQL track?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I’m on the fence. It’s not a great fit since we’re wrangling JSON at the end of the day and not SQL data. The input data makes sense to me as a data structure but not a table of data. There are a limited number of ways to solve it too.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My opinion is, since this is an SQLite track, we're trying to teach about its features, so doesn't it make sense to have an exercise where you practice handling JSON? Because JSON is actually used to store values in databases.

exercises/practice/etl/etl_test.sql Show resolved Hide resolved
exercises/practice/etl/etl_test.sql Show resolved Hide resolved
@IsaacG IsaacG added x:action/create Work on something from scratch x:size/large Large amount of work x:rep/large Large amount of reputation x:module/practice-exercise Work on Practice Exercises labels Jun 8, 2024
@Steffan153
Copy link
Contributor Author

Btw you don't need to mark these as x:size/large, they take me about 10 minutes lol

@vaeng
Copy link
Contributor

vaeng commented Jun 10, 2024

Btw you don't need to mark these as x:size/large, they take me about 10 minutes lol

It is mostly based on the exercism recommendation for exercise contribution. But if you want less rep, you can have it :D

@vaeng vaeng merged commit 02bde12 into exercism:main Jun 11, 2024
2 checks passed
@Steffan153 Steffan153 deleted the etl branch June 12, 2024 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x:action/create Work on something from scratch x:module/practice-exercise Work on Practice Exercises x:rep/large Large amount of reputation x:size/large Large amount of work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants