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

Add snowflake #1341

Open
wants to merge 103 commits into
base: master
Choose a base branch
from
Open

Add snowflake #1341

wants to merge 103 commits into from

Conversation

sergeylanzman
Copy link
Collaborator

No description provided.

alldoami and others added 30 commits October 9, 2019 16:38
@alldoami
draft
d53eab3
@alldoami
changes
fccd405
@alldoami
new files
18af926
@alldoami
renamed driver
c11ee4a
@alldoami
changed names
8d3dbd4
@alldoami
print statements for debug
b67fb5a
Some more fixes
7b3a748
@alldoami
removed provider from driver name
7caa592
@alldoami
updated snowflake name and renamed snowflake_user
27cdc64
@alldoami
remove comments
3912a59
@alldoami
removed unnecessary if
46c5775
@alldoami
removed comment
c8553b5
@alldoami
Update providers/snowflake/snowflake_client.go
afb6223 
Co-Authored-By: Eduardo Lopez <[email protected]>
@alldoami
removed if
ed8af6b
@alldoami
removed if
74f108a
@alldoami
fixes
a491b99
@alldoami
flattened error
f109d55
Update cmd/snowflake.go
a38d6bd
@alldoami
updated readme
c4bcd72
@alldoami
Merge branch 'adoami/import-databases' of github.com:chanzuckerberg/t…
bca5c5c 
…erraformer into adoami/import-databases
@alldoami
Merge pull request #2 from chanzuckerberg/adoami/import-databases
3157b3c 
Import Databases from Snowflake
@alldoami
importing database grants
beec652
@alldoami
changed names
b02eb85
@alldoami
database grants
fce59bb
@alldoami
db grant privilege bug go around
0129b96
@alldoami
Update providers/snowflake/database_grant.go
e840d68 
Co-Authored-By: Eduardo Lopez <[email protected]>
@alldoami
Merge pull request #3 from chanzuckerberg/adoami/import-database-grants
36104af
@alldoami
changed env var name from user to username
06cd1ed
@alldoami
Merge pull request #4 from chanzuckerberg/adoami/change-user-to-username
bd80781
Revert "Merge pull request #4 from chanzuckerberg/adoami/change-user-…
5b06812 
…to-username"

This reverts commit bd80781, reversing
changes made to 36104af.
alldoami and others added 23 commits November 4, 2019 14:04
@sergeylanzman sergeylanzman mentioned this pull request Jun 4, 2022
Closed
@bencohen2
Copy link

bencohen2 commented Jun 8, 2022 •
edited

@sergeylanzman I can add some commits here to fix the errors, if that's what you're waiting for. I'm also happy to test if you have any specific instructions on testing, though I've poked a bit around the repo and don't see anything.
I signed the CLA Agreement but still can't push to the repo - how long does it usually take to grant access?

@bencohen2
Copy link

@sergeylanzman Following up here. Submitted my CLA agreement but still don't have write access.

@bencohen2 bencohen2 mentioned this pull request Jun 17, 2022
Closed
@github-actions github-actions bot added the Stale label Aug 16, 2022
@github-actions github-actions bot added the Stale label Apr 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@sergeylanzman @bencohen2 @alldoami