Skip to content

Adapt sql for postgres#283

Closed
anphung wants to merge 1 commit intoquickfixgo:masterfrom
anphung:fix/postgres
Closed

Adapt sql for postgres#283
anphung wants to merge 1 commit intoquickfixgo:masterfrom
anphung:fix/postgres

Conversation

@anphung
Copy link

@anphung anphung commented May 23, 2017

  • Adapt sql for postgres placeholder ? vs $1

@cbusbey
Copy link
Contributor

cbusbey commented Jun 14, 2017

Hi @anphung,

First of all, apologies for the late response. I try to respond to requests asap, this one got away from me.

It is unfortunate that the sql driver placeholders are not abstracted... In the best interests of longer term planning to support a range of db impls, instead of a conditional block based on the underlying driver, we would rather either use a dep to handle this abstraction, or handle it internally with our own polymorphism (probably the latter). I'll create an issue so that we can track this to be fixed in master.

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.

2 participants