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

Support prepared statements with connection pooling #19

Merged
merged 1 commit into from
Jun 5, 2017
Merged

Conversation

irar2
Copy link
Contributor

@irar2 irar2 commented Jun 5, 2017

Kitura/Swift-Kuery#82

Support prepared statements with connection pooling for PostgreSQL by keeping a set of prepared statements for each connection and creating a prepared statement on the database if necessary

@codecov-io
Copy link

Codecov Report

Merging #19 into master will decrease coverage by 1.01%.
The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
- Coverage   73.69%   72.68%   -1.02%     
==========================================
  Files           3        3              
  Lines         422      432      +10     
==========================================
+ Hits          311      314       +3     
- Misses        111      118       +7
Flag Coverage Δ
#SwiftKueryPostgreSQL 72.68% <50%> (-1.02%) ⬇️
Impacted Files Coverage Δ
...es/SwiftKueryPostgreSQL/PostgreSQLConnection.swift 60.5% <50%> (-1.34%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2496a56...ca9a8c4. Read the comment docs.

@shmuelk
Copy link
Collaborator

shmuelk commented Jun 5, 2017

👍

@shmuelk shmuelk merged commit 3a89bc5 into master Jun 5, 2017
@shmuelk shmuelk deleted the issue_82 branch June 5, 2017 08:29
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.

3 participants