dbt-labs / dbt-spark

dbt-spark contains all of the code enabling dbt to work with Apache Spark and Databricks
https://getdbt.com
Apache License 2.0
395 stars 221 forks source link

Update `standardize_grants_dict` to handle potential case mismatches in grants table response #1087

Open lamalex opened 1 month ago

lamalex commented 1 month ago

Match case to column names returned rather than hardcoding. Fixes: #1086

resolves #1086 docs dbt-labs/docs.getdbt.com/#

Problem

DBT crashing due to column case mismatch in response from databricks on show grants statement.

Solution

Match column name case to response rather than hardcoding names.

Checklist

cla-bot[bot] commented 1 month ago

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR.

CLA has not been signed by users: @lamalex

lamalex commented 1 month ago

Signed!

cla-bot[bot] commented 1 month ago

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR.

CLA has not been signed by users: @lamalex

lamalex commented 1 month ago

signed again!