This repository was archived by the owner on Dec 30, 2019. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 64
Merged
2.0 #67
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
e2bf705
Initial work
brianc d3c86fe
Make progress on custom pool
brianc 0f6e85d
Make all original tests pass
brianc 3cc72ce
Fix test race
brianc 1581df3
Fix test when DNS is missing
brianc 2205dd8
Test more error conditions
brianc 3d8a9da
Add test for byop
brianc a3de64e
Add BYOP tests for errors
brianc 7d6a189
Add test for idle client error expunging
brianc a85e9ec
Fix typo
brianc c970076
Replace var with const/let
brianc 0b421c0
Remove var usage
brianc 20c4c34
Fix linting
brianc cb7b8e6
Work on connection timeout
brianc 8b9e1fb
Work on error condition tests
brianc 93dcadb
Remove logging
brianc 4d5bac8
Add connection timeout
brianc 947d7a8
Add idle timeout
brianc 87d657d
Test for returning to client to pool after error
brianc 7d9b22a
Add idleTimeout support to native client
brianc 2085a6d
Add pg as peer dependency
brianc f356bae
Rename properties
brianc e1b6ea9
Fix lint
brianc be8d58b
use strict
brianc 36f1c9a
Add draining to pool.end
brianc bfd4f37
Ensure ending pools drain properly
brianc 4af9ad4
Remove yarn.lock
brianc bfa7ae7
Remove object-assign
brianc 024795e
Remove node 8
brianc 6a2a302
Remove closure for waiter construction
brianc 0692b96
Ensure client.connect is never sync
brianc e444519
Fix lint
brianc fc6c2de
Change to es6 class
brianc cbdbccc
Code cleanup & lint fixes
brianc File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,6 +8,3 @@ matrix: | |
| - node_js: "6" | ||
| addons: | ||
| postgresql: "9.4" | ||
| - node_js: "8" | ||
| addons: | ||
| postgresql: "9.4" | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just wonder : why remove node 8 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it was messing up in travis but working locally even though the same tests were passing locally - I just removed it temporarily so I could make sure the other tests were passing. I'll need to get it working in travis eventually.