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

Rebasing develop for latest 0.21.0 RC release #216

Merged
merged 15 commits into from
Sep 17, 2021

Conversation

leahwicz
Copy link
Contributor

Rebase to capture all changes to develop since last beta release for 0.21.0

JCZuurmond and others added 14 commits August 9, 2021 21:41
* Use exception chaining to get more detailed feedback when pyodbc is not installed

* Remove pyodbc referenced before assignment

* Set back try except

* Add flake ignore

* Add error message to RunTimeException

Error chaining does not show the message in `dbt debug`. Therefore we
explicitly add the message to the dbt.exceptions.RunTimeException

* Update change log

Add to change log that we print the import error when pyodbc can not be imported

* Fix parenthesis in change log
* Add support for ODBC Server Side Parameters

* Update CHANGELOG
* Code changes

* README changes

* Improve error message default

* Changelog

* Changelog corrections

* Restore accidental deletion

* Update dbt/adapters/spark/connections.py

Co-authored-by: Jeremy Cohen <jtcohen6@gmail.com>

* Add myself to Contributors

Co-authored-by: Jeremy Cohen <jtcohen6@gmail.com>
* fixed get_columns_in_relation for open source delta table

* fixed E501 linting error and added change log
* fix issue parsing structs

* include contributor in changelog

* better error explanation

Co-authored-by: Jeremy Cohen <jeremy@fishtownanalytics.com>
* Add adapter unique_field

* Fix flake8. Add changelog entry
* fix: requirements.txt to reduce vulnerabilities


The following vulnerabilities are fixed by pinning transitive dependencies:
- https://snyk.io/vuln/SNYK-PYTHON-SQLPARSE-1584201
- https://snyk.io/vuln/SNYK-PYTHON-THRIFT-474615

* Removing Thrift conflict with versions over 12

Co-authored-by: leahwicz <60146280+leahwicz@users.noreply.github.com>
@cla-bot
Copy link

cla-bot bot commented Sep 15, 2021

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Leah Antkiewicz.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email email@example.com
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@cla-bot
Copy link

cla-bot bot commented Sep 15, 2021

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Leah Antkiewicz.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email email@example.com
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@leahwicz leahwicz requested a review from jtcohen6 September 15, 2021 17:35
Copy link
Contributor

@jtcohen6 jtcohen6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CLA is failing because one of the previous commits is by a Leah lookalike. Not a huge deal, we can still merge.

Edit: Weirdly, I don't see the changes associated with the v0.21.0b2 version bump anymore. Not a huge deal as long as the release workflow can still handle the bump to v0.21.0rc1.

@leahwicz leahwicz merged commit 4607be4 into 0.21.latest Sep 17, 2021
@mikealfare mikealfare deleted the leahwicz/0.21.0-rebase branch March 1, 2023 00:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants