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

[Improvement] Spark Connector Need DelegationTokenProvider for k8s deployment #3297

Open
theoryxu opened this issue May 7, 2024 · 1 comment
Labels
improvement Improvements on everything

Comments

@theoryxu
Copy link
Contributor

theoryxu commented May 7, 2024

What would you like to be improved?

When deploying a spark application on k8s and then connecting multiple Hive Metastore (cluster mode), the spark needs DelegationTokenProvider to get delegate tokens from different HMS in the submitting stage and store them at the UserGroupInformation so that the spark driver can communicate with HMS.

For example, KyuubiHiveConnector contains the KyuubiHiveConnectorDelegationTokenProvider to deal with this case.

Now, the Gravitino Spark Connector depends on the KyuubiHiveConnector. However, the KyuubiHiveConnectorDelegationTokenProvider filters the catalog's implementation, which doesn't work in the above case. In addition, It is only for the hive catalog, not including the iceberg catalog.

The Gravitino Spark Connector needs its DelegationTokenProvider to handle this case and ensure it works well in both hive and iceberg catalogs under a Kerberos environment.

REF:
https://github.com/apache/kyuubi/blob/master/extensions/spark/kyuubi-spark-connector-hive/src/main/scala/org/apache/kyuubi/spark/connector/hive/KyuubiHiveConnectorDelegationTokenProvider.scala

apache/kyuubi#4560

How should we improve?

No response

@theoryxu theoryxu added the improvement Improvements on everything label May 7, 2024
@theoryxu theoryxu changed the title [Improvement] Spark Connector Need DelegationTokenProvider [Improvement] Spark Connector Need DelegationTokenProvider for k8s deployment May 7, 2024
@FANNG1
Copy link
Contributor

FANNG1 commented May 7, 2024

cc @danhuawang ,could our test cluster cover the scene?

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

No branches or pull requests

2 participants