Skip to content

Lightweight support for Postgresql #404

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 30, 2020
Merged

Lightweight support for Postgresql #404

merged 2 commits into from
Sep 30, 2020

Conversation

daemonl
Copy link
Contributor

@daemonl daemonl commented May 29, 2020

Addresses #284 without an overhaul

  • Adds a function to replace ? placeholders with $1 $2 etc as required by postgres
  • Runs all sql statements through the replacement func at query time (nop for sql)
  • In theory any other placeholder format could be added pretty easily on top

@sami-sweng sami-sweng mentioned this pull request Sep 2, 2020
Open
@cbusbey
Copy link
Contributor

cbusbey commented Sep 30, 2020

Thanks for the contribution! 🍻

@cbusbey cbusbey merged commit 72532ac into quickfixgo:master Sep 30, 2020
@jeffwidman jeffwidman mentioned this pull request May 19, 2021
Closed
level2player pushed a commit to longbridge/quickfix that referenced this pull request Jul 19, 2022
@cbusbey
Merge pull request quickfixgo#404 from EmbedFi/master
2eae3eb 
Lightweight support for Postgresql
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@daemonl @cbusbey