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 snowflake support #24

Closed
wants to merge 2 commits into from

Conversation

ssilwal29
Copy link

@ssilwal29 ssilwal29 commented Sep 9, 2020

Hi! thanks for starting this project. its really neat. I have attempted to add snowflake support and wondering if you are accepting any pull requests. I am new to Golang and tried to follow your code as much as possible. Let me know if something are not upto the standards. I will also add tests and to readme if you are fine adding this to the repo.

Thanks

@lighttiger2505
Copy link
Collaborator

Thanks for the report. I understand that you want to add snowflake support to sqls, and I'm very grateful that you made a pull request for it.

There are a number of reasons for this, but the main problem is that it can be difficult to maintain. Especially with products like Snowflake, BigQuery and Athena, for which I can't easily set up a test environment, I can't make sure that the code is correct. This is critical for me to maintain.

@ssilwal29
Copy link
Author

yeah that makes sense. i think you can sign up for a trial snowflake account but dont think its ideal. I think I can try to maintain this on the fork unless if you have other ideas. thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants