kevinjqliu closed pull request #766: Implement Kerberos authentication support
for Hive Catalog
URL: https://github.com/apache/iceberg-python/pull/766
--
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
kevinjqliu commented on PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#issuecomment-2654704906
THanks @yothinix for the contribution, this is merged at #1634
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
Fokko commented on code in PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#discussion_r1948746839
##
pyiceberg/catalog/hive.py:
##
@@ -138,17 +141,34 @@ class _HiveClient:
_client: Client
_ugi: Optional[List[str]]
-def __init__(self, uri: str, ugi
kevinjqliu commented on PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#issuecomment-2628602907
hi @yothinix thanks for the contribution. Would love like to address the
comment above? Would be great to have this as part of the 0.9.0 release
--
This is an automated message
uniqueinput commented on PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#issuecomment-2436364333
It would be really great if this could make it into 0.8. :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
kevinjqliu commented on code in PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#discussion_r1740136965
##
mkdocs/docs/configuration.md:
##
@@ -268,19 +268,15 @@ catalog:
catalog:
default:
uri: thrift://localhost:9083
+hive.hive2-compatible: true
+
yothinix commented on PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#issuecomment-2294533715
Hi @kevinjqliu Thank you for look into this PR, I just rebased the PR branch
to latest main branch as requested.
--
This is an automated message from the Apache Git Service.
To re
kevinjqliu commented on PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#issuecomment-2278296348
Hi @yothinix do you mind rebasing with main? I think this is very close to
being merged, I can work with you to get it through.
--
This is an automated message from the Apa
yothinix commented on PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#issuecomment-2160907957
Hi @Fokko I added new change as commented, could you help review it again
Also, As I tested more on HMS behind Kerberize we found out that it's
required the thrift client to b
yothinix commented on code in PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#discussion_r1634984001
##
mkdocs/docs/configuration.md:
##
@@ -228,19 +228,19 @@ catalog:
catalog:
default:
uri: thrift://localhost:9083
-s3.endpoint: http://localhost:9000
yothinix commented on code in PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#discussion_r163498
##
mkdocs/docs/configuration.md:
##
@@ -228,19 +228,19 @@ catalog:
catalog:
default:
uri: thrift://localhost:9083
-s3.endpoint: http://localhost:9000
yothinix commented on code in PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#discussion_r1634979621
##
mkdocs/docs/configuration.md:
##
@@ -228,19 +228,19 @@ catalog:
catalog:
default:
uri: thrift://localhost:9083
-s3.endpoint: http://localhost:9000
Fokko commented on code in PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#discussion_r1632564486
##
mkdocs/docs/configuration.md:
##
@@ -228,19 +228,19 @@ catalog:
catalog:
default:
uri: thrift://localhost:9083
-s3.endpoint: http://localhost:9000
-
yothinix opened a new pull request, #766:
URL: https://github.com/apache/iceberg-python/pull/766
## Change proposed
- Introduce support for HMS behind kerberize environment, to enable this
feature we need to add catalog config `hive.use-kerberos=true`
Related to #135
--
This is
14 matches
Mail list logo
