Skip to content

db: reconnect on connection errors - #286

Open
xiam wants to merge 30 commits into
masterfrom
feature/test-reconnect-with-few-connections
Open

db: reconnect on connection errors#286
xiam wants to merge 30 commits into
masterfrom
feature/test-reconnect-with-few-connections

Conversation

@xiam

@xiamxiam commented Nov 10, 2016

Copy link
Copy Markdown
Member

Fixes after testing reconnect with a low number of available connections to keep everything smooth.

José Carlos Nietoand others added 30 commits October 4, 2016 11:09
José Carlos Nieto
Add a recovering mechanism to reconnect and retry a query if the serv…
9012927
…er is restarting or refusing connections.
José Carlos Nieto
Fix bug with ghost values in execer
cabedd7
José Carlos Nieto
Update retry mechanism.
c93dbcf
José Carlos Nieto
Merge branch 'issue-226' of github.com:upper/db into issue-226
1de668f
José Carlos Nieto
Make d.recoverFromErr only catch ErrBadConn.
6475aea
José Carlos
Merge pull request#282from upper/issue-280
b3306f0
Add default AND join within Or. Closes#280
José Carlos
Merge pull request#281from upper/issue-279
c267726
Fix handling of slice parameters. Closes#279
José Carlos Nieto
Manual merge
de7bff7
José Carlos Nieto
Add reconnect (WIP +debug messages)
7a86715
José Carlos Nieto
Better logic for handling connect/reconnect
0f9bfe3
José Carlos Nieto
Cleaning up remaining reconnection bits
4af4871
José Carlos Nieto
Removing log messages and adding a test for bad connection.
f08946e
José Carlos Nieto
Update docstrings
e523258
José Carlos Nieto
Remove "could not read block" from db.ErrServerRefusedConnection
2a02240
José Carlos Nieto
Ping within a transaction.
cfb43e3
José Carlos Nieto
Don't block Transaction() before connecting.
3538ba3
José Carlos
Merge pull request#269from upper/issue-226
13181c0
Add a recovering mechanism to reconnect when required
José Carlos Nieto
Use db.ErrTooManyReconnectionAttempts instead of db.ErrGivingUpTrying…
ec0678d
…ToConnect
José Carlos Nieto
Merge branch 'master' into feature/err-too-many-reconnection-attempts
6d206db
José Carlos
Merge pull request#283from upper/feature/err-too-many-reconnection-…
53a97dd
…attempts
Feature/err too many reconnection attempts
José Carlos Nieto
Fixes after testing reconnect with a low number of available connecti…
37e2780
…ons.
José Carlos Nieto
Make retry-on-error configurable and disable it by default.
78d28bc
José Carlos Nieto
Add db.Conf.SetRetryQueryOnError to PostgreSQL test.
340d8eb
José Carlos Nieto
Fix call to c.setBinaryOption
118944d
José Carlos Nieto
Add missing db.Conf.RetryQueryOnError
7017914
José Carlos Nieto
Add d.recoverAttempts for counting recovery attempts up to discardCon…
ed13348
…nectionAfterFailedRecoverAttempts times
José Carlos Nieto
Use flushConnectionPoolAfterRecoverAttempts and remove ConnCheck.
02b5a72
José Carlos Nieto
Merge branch 'master' into feature/test-reconnect-with-few-connections
963d413
José Carlos Nieto
Removing flawed logic: don't close before having a valid new conn to …
838332f
…replace the old one
José Carlos Nieto
Mark prepared statements as ready to be closed after using them
33b8d62
@xiamxiam added the icebox label Nov 24, 2016
@xiamxiam changed the title Reconnect with a low number of available connectionsReconnect on connection errorsNov 24, 2016
@xiamxiam added the wip label Dec 9, 2016
@xiamxiam changed the title Reconnect on connection errorsdb: reconnect on connection errorsJul 20, 2017
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

iceboxwip

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@xiam