Re: [PR] Glue: Allow for assuming role for Glue [iceberg-python]

2025-02-27 Thread via GitHub
Fokko commented on PR #1299: URL: https://github.com/apache/iceberg-python/pull/1299#issuecomment-2689007720 Since we undeprecated botocore session, we can close this one -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] Glue: Allow for assuming role for Glue [iceberg-python]

2025-02-27 Thread via GitHub
Fokko closed pull request #1299: Glue: Allow for assuming role for Glue URL: https://github.com/apache/iceberg-python/pull/1299 -- 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.

Re: [PR] Glue: Allow for assuming role for Glue [iceberg-python]

2024-11-06 Thread via GitHub
kevinjqliu commented on code in PR #1299: URL: https://github.com/apache/iceberg-python/pull/1299#discussion_r1831454666 ## pyiceberg/io/__init__.py: ## @@ -61,7 +61,7 @@ AWS_SECRET_ACCESS_KEY = "client.secret-access-key" AWS_SESSION_TOKEN = "client.session-token" AWS_ROLE_AR

Re: [PR] Glue: Allow for assuming role for Glue [iceberg-python]

2024-11-06 Thread via GitHub
kevinjqliu commented on code in PR #1299: URL: https://github.com/apache/iceberg-python/pull/1299#discussion_r1831452825 ## pyiceberg/catalog/glue.py: ## @@ -296,13 +306,48 @@ class GlueCatalog(MetastoreCatalog): def __init__(self, name: str, **properties: Any): su

Re: [PR] Glue: Allow for assuming role for Glue [iceberg-python]

2024-11-06 Thread via GitHub
cshenrik commented on code in PR #1299: URL: https://github.com/apache/iceberg-python/pull/1299#discussion_r1831031255 ## pyiceberg/catalog/glue.py: ## @@ -296,13 +306,48 @@ class GlueCatalog(MetastoreCatalog): def __init__(self, name: str, **properties: Any): supe

Re: [PR] Glue: Allow for assuming role for Glue [iceberg-python]

2024-11-06 Thread via GitHub
Fokko commented on code in PR #1299: URL: https://github.com/apache/iceberg-python/pull/1299#discussion_r1831238850 ## pyiceberg/catalog/glue.py: ## @@ -296,13 +306,48 @@ class GlueCatalog(MetastoreCatalog): def __init__(self, name: str, **properties: Any): super()

Re: [PR] Glue: Allow for assuming role for Glue [iceberg-python]

2024-11-06 Thread via GitHub
Fokko commented on code in PR #1299: URL: https://github.com/apache/iceberg-python/pull/1299#discussion_r1831238850 ## pyiceberg/catalog/glue.py: ## @@ -296,13 +306,48 @@ class GlueCatalog(MetastoreCatalog): def __init__(self, name: str, **properties: Any): super()

Re: [PR] Glue: Allow for assuming role for Glue [iceberg-python]

2024-11-06 Thread via GitHub
cshenrik commented on code in PR #1299: URL: https://github.com/apache/iceberg-python/pull/1299#discussion_r1831036603 ## pyiceberg/catalog/glue.py: ## @@ -296,13 +306,48 @@ class GlueCatalog(MetastoreCatalog): def __init__(self, name: str, **properties: Any): supe