Re: [PR] Add Snowflake catalog [iceberg-python]

2024-10-14 Thread via GitHub
prabodh1194 closed pull request #687: Add Snowflake catalog URL: https://github.com/apache/iceberg-python/pull/687 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscrib

Re: [PR] Add Snowflake catalog [iceberg-python]

2024-10-14 Thread via GitHub
prabodh1194 commented on PR #687: URL: https://github.com/apache/iceberg-python/pull/687#issuecomment-2412914581 closing as I won't be able to continue with this PR now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Add Snowflake catalog [iceberg-python]

2024-10-14 Thread via GitHub
ccollingwoodAutodesk commented on code in PR #687: URL: https://github.com/apache/iceberg-python/pull/687#discussion_r1799807246 ## pyiceberg/catalog/snowflake_catalog.py: ## @@ -0,0 +1,289 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor l

Re: [PR] Add Snowflake catalog [iceberg-python]

2024-10-14 Thread via GitHub
ccollingwoodAutodesk commented on code in PR #687: URL: https://github.com/apache/iceberg-python/pull/687#discussion_r1799807246 ## pyiceberg/catalog/snowflake_catalog.py: ## @@ -0,0 +1,289 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor l

Re: [PR] Add Snowflake catalog [iceberg-python]

2024-07-13 Thread via GitHub
prabodh1194 commented on PR #687: URL: https://github.com/apache/iceberg-python/pull/687#issuecomment-2226959566 @Fokko , considering that snowflake's rest catalog docs are still not released to public, we can only wait for it. However, I believe that this PR will continue to be relevant co

Re: [PR] Add Snowflake catalog [iceberg-python]

2024-07-05 Thread via GitHub
Fokko commented on PR #687: URL: https://github.com/apache/iceberg-python/pull/687#issuecomment-2210949950 @prabodh1194 Thanks for working on this. What are your thoughts on Snowflake adding REST catalog support, would this still be needed? Could you also make sure that you add this c

Re: [PR] Add Snowflake catalog [iceberg-python]

2024-07-05 Thread via GitHub
Fokko commented on code in PR #687: URL: https://github.com/apache/iceberg-python/pull/687#discussion_r1666855752 ## pyiceberg/catalog/snowflake_catalog.py: ## @@ -0,0 +1,289 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreemen

Re: [PR] Add Snowflake catalog [iceberg-python]

2024-07-05 Thread via GitHub
Fokko commented on code in PR #687: URL: https://github.com/apache/iceberg-python/pull/687#discussion_r1666852105 ## pyiceberg/catalog/snowflake_catalog.py: ## @@ -0,0 +1,289 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreemen

Re: [PR] Add Snowflake catalog [iceberg-python]

2024-06-08 Thread via GitHub
prabodh1194 commented on code in PR #687: URL: https://github.com/apache/iceberg-python/pull/687#discussion_r1632063906 ## pyiceberg/catalog/snowflake_catalog.py: ## @@ -0,0 +1,237 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license ag

Re: [PR] Add Snowflake catalog [iceberg-python]

2024-06-01 Thread via GitHub
prabodh1194 commented on code in PR #687: URL: https://github.com/apache/iceberg-python/pull/687#discussion_r1623241664 ## pyiceberg/catalog/snowflake_catalog.py: ## @@ -0,0 +1,237 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license ag