-
Notifications
You must be signed in to change notification settings - Fork 233
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 lib/pq with pgx #627
Conversation
What do you think if it’s possible to remove dependency on lib/pq? I noticed here it’s still being used for custom types — https://github.com/upper/db/blob/issue-621_pgx-driver-for-postgresql/adapter/postgresql/custom_types.go#L30 perhaps that is fine though? Or.. should we update the custom type code to use pgx.. it would require a lot of what lib/pq does to move here, I’m not sure it makes a difference .. lmk what you think |
@pkieltyka Yes, that's for now because we're using those types for tests, but the goal is to remove all lib/pq dependencies. |
8af4bb1
to
2dac4f3
Compare
…s and tools for JSONB conversion
4ce5abd
to
c165aee
Compare
This has been working fine for a while. Let's merge into master, we'll include this on the next release. |
This PR replaces the lib/pq driver with the pgx driver. It also changes code around automatic type conversions which was required to facilitate said migration.