The Wayback Machine - https://web.archive.org/web/20210204230027/https://github.com/encode/databases/pull/204
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

Replace psycopg2-binary with psycopg2 (#198) #204

Merged
merged 1 commit into from May 1, 2020

Conversation

@retnikt
Copy link
Contributor

@retnikt retnikt commented May 1, 2020

No description provided.

Copy link
Member

@tomchristie tomchristie left a comment

Yup, thanks.

@tomchristie tomchristie merged commit a411e44 into encode:master May 1, 2020
3 checks passed
3 checks passed
Python 3.6
Details
Python 3.7
Details
Python 3.8
Details
@retnikt retnikt deleted the retnikt:patch-1 branch May 1, 2020
@JayH5
Copy link
Member

@JayH5 JayH5 commented May 1, 2020

Just a drive-by comment that this change probably needs a bit of a warning in the changelog as the psycopg2 package requires build tools, libpq, etc, whereas before the psycopg2-binary package did not.

@tomchristie
Copy link
Member

@tomchristie tomchristie commented May 1, 2020

Okay, which makes me want to re-evaluate what benefit this change would give us.

@retnikt
Copy link
Contributor Author

@retnikt retnikt commented May 1, 2020

Well, really I think users should have the choice between the binary or non-binary version depending on their circumstances. The only option for this I believe would be to remove the requirement entirely and tell users in the docs that one of them is required for table creation/migrations. This also allows the flexibility of running your migrations entirely separately from your web server.

@tomchristie
Copy link
Member

@tomchristie tomchristie commented May 1, 2020

Then let's drop psycopg2 and pymysql entirely, and make sure that the next version is a proper version bump, and calls out the requirement to install a sync driver if you're using alembic or SQL Alchemy's engine.create_all()

retnikt added a commit to retnikt/databases that referenced this pull request May 1, 2020
retnikt added a commit to retnikt/databases that referenced this pull request May 1, 2020
TODO: set release information
vmarkovtsev added a commit to vmarkovtsev/databases that referenced this pull request Oct 19, 2020
Changelog:

- Use backend native fetch_val() implementation when available (encode#132)
- Replace psycopg2-binary with psycopg2 (encode#198) (encode#204)
- Speed up PostgresConnection fetch() and iterate() (encode#193)
- Access asyncpg Record field by key on raw query (encode#207)
- Fix type hinting for sqlite backend (encode#227)
- Allow setting min_size and max_size in postgres DSN (encode#210)
- Add option pool_recycle in postgres DSN (encode#233)
- Fix SQLAlchemy DDL statements (encode#226)
- Make fetch_val call fetch_one for type conversion (encode#246)
- Allow extra transaction options (encode#242)
- Unquote username and password in Databaseurl(encode#248)
vmarkovtsev added a commit to vmarkovtsev/databases that referenced this pull request Oct 19, 2020
Changelog:

- Use backend native fetch_val() implementation when available (encode#132)
- Replace psycopg2-binary with psycopg2 (encode#198) (encode#204)
- Speed up PostgresConnection fetch() and iterate() (encode#193)
- Access asyncpg Record field by key on raw query (encode#207)
- Fix type hinting for sqlite backend (encode#227)
- Allow setting min_size and max_size in postgres DSN (encode#210)
- Add option pool_recycle in postgres DSN (encode#233)
- Fix SQLAlchemy DDL statements (encode#226)
- Make fetch_val call fetch_one for type conversion (encode#246)
- Allow extra transaction options (encode#242)
- Unquote username and password in Databaseurl(encode#248)
@vmarkovtsev vmarkovtsev mentioned this pull request Oct 19, 2020
vmarkovtsev added a commit to vmarkovtsev/databases that referenced this pull request Oct 20, 2020
Changelog:

- Use backend native fetch_val() implementation when available (encode#132)
- Replace psycopg2-binary with psycopg2 (encode#198) (encode#204)
- Speed up PostgresConnection fetch() and iterate() (encode#193)
- Access asyncpg Record field by key on raw query (encode#207)
- Fix type hinting for sqlite backend (encode#227)
- Allow setting min_size and max_size in postgres DSN (encode#210)
- Add option pool_recycle in postgres DSN (encode#233)
- Fix SQLAlchemy DDL statements (encode#226)
- Make fetch_val call fetch_one for type conversion (encode#246)
- Allow extra transaction options (encode#242)
- Unquote username and password in Databaseurl(encode#248)
vmarkovtsev added a commit to vmarkovtsev/databases that referenced this pull request Oct 20, 2020
Changelog:

- Use backend native fetch_val() implementation when available (encode#132)
- Replace psycopg2-binary with psycopg2 (encode#198) (encode#204)
- Speed up PostgresConnection fetch() and iterate() (encode#193)
- Access asyncpg Record field by key on raw query (encode#207)
- Fix type hinting for sqlite backend (encode#227)
- Allow setting min_size and max_size in postgres DSN (encode#210)
- Add option pool_recycle in postgres DSN (encode#233)
- Fix SQLAlchemy DDL statements (encode#226)
- Make fetch_val call fetch_one for type conversion (encode#246)
- Allow extra transaction options (encode#242)
- Unquote username and password in Databaseurl(encode#248)
vmarkovtsev added a commit that referenced this pull request Oct 20, 2020
Changelog:

- Use backend native fetch_val() implementation when available (#132)
- Replace psycopg2-binary with psycopg2 (#198) (#204)
- Speed up PostgresConnection fetch() and iterate() (#193)
- Access asyncpg Record field by key on raw query (#207)
- Fix type hinting for sqlite backend (#227)
- Allow setting min_size and max_size in postgres DSN (#210)
- Add option pool_recycle in postgres DSN (#233)
- Fix SQLAlchemy DDL statements (#226)
- Make fetch_val call fetch_one for type conversion (#246)
- Allow extra transaction options (#242)
- Unquote username and password in Databaseurl(#248)
@vmarkovtsev
Copy link
Contributor

@vmarkovtsev vmarkovtsev commented Oct 21, 2020

Here I am, the one who released 0.4.0 and broke my own production builds because of this 😓

So shall I remove these requirements completely then?

vmarkovtsev added a commit to vmarkovtsev/databases that referenced this pull request Oct 21, 2020
vmarkovtsev added a commit to vmarkovtsev/databases that referenced this pull request Oct 21, 2020
vmarkovtsev added a commit to vmarkovtsev/databases that referenced this pull request Oct 21, 2020
vmarkovtsev added a commit that referenced this pull request Nov 16, 2020
Related to #204
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
HTTPS · web.archive.org
← Home