Skip to content

Refactored connection pool #913

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

Merged
merged 14 commits into from
Jul 26, 2019
Merged

Refactored connection pool #913

merged 14 commits into from
Jul 26, 2019

Conversation

delvedor
Copy link
Member

  • Created BaseConnectionPool class
  • Created CloudConnectionPool
  • connection pool updates are immutable
  • resurrect now happens inside getConnection()
  • Rewritten connection pool(s) type definitions

delvedor added 3 commits July 18, 2019 18:58
- Created BaseConnectionPool class
- Created CloudConnectionPool
- connection pool updates are immutable
- resurrect now happens inside getConnection()
@delvedor delvedor merged commit a948a98 into master Jul 26, 2019
@delvedor delvedor deleted the refactor-connection-pool branch July 26, 2019 09:43
delvedor added a commit that referenced this pull request Jul 26, 2019
* Refactored ConnectionPool
- Created BaseConnectionPool class
- Created CloudConnectionPool
- connection pool updates are immutable
- resurrect now happens inside getConnection()

* Rewritten connection pool(s) type definitions

* Updated test

* Fixed test

* Fix if check

* Removed old files

* Improve code coverage

* Updated license header

* Fix if check

* Improve code coverage

* Updated coverage script
delvedor added a commit that referenced this pull request Jul 26, 2019
* Refactored ConnectionPool
- Created BaseConnectionPool class
- Created CloudConnectionPool
- connection pool updates are immutable
- resurrect now happens inside getConnection()

* Rewritten connection pool(s) type definitions

* Updated test

* Fixed test

* Fix if check

* Removed old files

* Improve code coverage

* Updated license header

* Fix if check

* Improve code coverage

* Updated coverage script
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.

1 participant