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

Bump/psycopg2 2.7.5 #807

Merged
merged 2 commits into from
Jun 28, 2018
Merged

Bump/psycopg2 2.7.5 #807

merged 2 commits into from
Jun 28, 2018

Conversation

drewbanin
Copy link
Contributor

Fixes #773, #704

We originally intended to use the psycopg2-binary dep, but it sounds like that's not ready for production workloads yet. Via the installation docs:

Note The -binary package is meant for beginners to start playing with Python and PostgreSQL without the need to meet the build requirements. If you are the maintainer of a publish package depending on psycopg2 you shouldn’t use psycopg2-binary as a module dependency. For production use you are advised to use the source distribution.

This PR bumps the dep to 2.7.5, which should alleviate issues with installation on Fedora/Arch/etc. It also contains fixes around the closed attribute on connections, which we will likely need to use to manage long-running queries that result in disconnected cursors.

@drewbanin drewbanin added the dependencies Changes to the version of dbt dependencies label Jun 28, 2018
@drewbanin drewbanin merged commit a382def into development Jun 28, 2018
@drewbanin drewbanin deleted the bump/psycopg2-2.7.5 branch June 28, 2018 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Changes to the version of dbt dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants