-
Notifications
You must be signed in to change notification settings - Fork 75
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
Convert DDL to Snowflake - Not an issue any more, just some code others might benefit from. #38
Comments
Thanks for the email.
You are correct that I should not use reserved words for column names. It’s something I've meant to change this.
Also, my box dimension puzzle answer is wrong. The answer gives a solution if they have the same volume, but not the same dimensions. I will need to update this as well.
Scott
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: jmadison222 ***@***.***>
Sent: Sunday, March 17, 2024 10:37:56 AM
To: smpetersgithub/AdvancedSQLPuzzles ***@***.***>
Cc: Subscribed ***@***.***>
Subject: Re: [smpetersgithub/AdvancedSQLPuzzles] Rename columns to remove brackets so Snowflake works better. (Issue #38)
Ah, wait! Never mind. This already happens in a bunch of places that I replaced, but there are apparently a few new ones. I'll just sed the script.
sed -i 's/\[/\"/g' puzz.sql
sed -i 's/\]/\"/g' puzz.sql
Easy enough.
Rock on!
James
—
Reply to this email directly, view it on GitHub<#38 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ANP5G72G76EK5NLSOBSM2GTYYW2FJAVCNFSM6AAAAABE2JPJIKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBSGUYTEMJQGI>.
You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
|
That would be great if you did the column rename. Also, I have a number of other observations for a number of questions around ambiguities and edge cases. As the months and years go by, I can submit some to you if you want. Not trying to make work if this is just a hobby, etc. But I study and work the problems carefully, so I'm happy to send along my thoughts on what might be good improvements. Then you can do with them what you will. Thanks! James |
Yes. Please share any findings. Im always thankful to get the puzzles peer reviewed by other developers.
Scott
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: jmadison222 ***@***.***>
Sent: Tuesday, March 19, 2024 9:36:45 AM
To: smpetersgithub/AdvancedSQLPuzzles ***@***.***>
Cc: Scott Peters ***@***.***>; Comment ***@***.***>
Subject: Re: [smpetersgithub/AdvancedSQLPuzzles] Convert DDL to Snowflake - Not an issue any more, just some code others might benefit from. (Issue #38)
That would be great if you did the column rename. Also, I have a number of other observations for a number of questions around ambiguities and edge cases. As the months and years go by, I can submit some to you if you want.
Not trying to make work if this is just a hobby, etc. But I study and work the problems carefully, so I'm happy to send along my thoughts on what might be good improvements. Then you can do with them what you will.
Thanks!
James
—
Reply to this email directly, view it on GitHub<#38 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ANP5G77CYSZVRWUTC3MYVELYZBEP3AVCNFSM6AAAAABE2JPJIKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBXGM2TIMJYHA>.
You are receiving this because you commented.Message ID: ***@***.***>
|
Scott:
Love your product! But forget this request. Short version, I use Snowflake, and I need to make some tweaks to get it there, but I'm good.
For reference, here is what I run to get on Snowflake:
Thanks!
James
The text was updated successfully, but these errors were encountered: