-
Notifications
You must be signed in to change notification settings - Fork 234
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
[CT-821] Allow passing through Cluster Configuration parameters via profiles.yml when using the thrift connection method #387
Comments
This issue has been marked as Stale because it has been open for 180 days with no activity. If you would like the issue to remain open, please remove the stale label or comment on the issue, or it will be closed in 7 days. |
Hi, |
Hey @hanna-liashchuk, could you please check this discussion, I'm trying to regroup all the threads on that topic in one place. |
Hi @VShkaberda , have you considered the SQL hints. For the repartition example in the issue I recommend the hints over setting the configuration as you might want to change this configuration parameter for different models. |
@JCZuurmond I appreciate your hint. My case is more common, e.g. passing |
Description
Allow passing through Cluster Configuration parameters via profiles.yml when using the thrift connection method.
This allows configuration of values that would need to be set via SET to be run in the Spark session and take effect on DBT models. Analogous feature for odbc has been implemented in this request.
Alternatives
Alternative is to use
pre_hook
(taken example) but this method doesn't work quite well and has disadvantages.This feature can be implemented through passing Configuration parameters as configuration parameter of
hive.connect()
of pyHive.Benefits
spark.hadoop.fs.s3a.access.key
,spark.hadoop.fs.s3a.secret.key
).Our team may be contributing this feature. One issue is the naming in the
profiles.yml
: should it beserver_side_parameters
to make it matching used inodbc
orconfiguration
to make it matching used inhive.connect()
. Or both name should be allowed?The text was updated successfully, but these errors were encountered: