Skip to content

Connection Pooling Support#571

Closed
ELD wants to merge 1 commit into
rwf2:masterfrom
ELD:connection-pool
Closed

Connection Pooling Support#571
ELD wants to merge 1 commit into
rwf2:masterfrom
ELD:connection-pool

Conversation

@ELD

@ELD ELD commented Feb 24, 2018 •
edited
Loading

Copy link
Copy Markdown
Member

There aren't any procedural macros yet, just a rough cut at the initial fairing implementation for multiple connections.

There are definitely holes here, just figured this was a good starting point to discuss how to proceed from here.

Things to do to merge this PR:

  • Procedural macro (#[database("name")])
  • Database config fetching
    • Refactor method
  • Rust postgres adapter
  • Diesel adapters (Sqlite, MySQL, Postgres)
  • Rust Redis adapter
  • Rust MySQL adapter
  • Rust Sqlite Adapter
  • Rust Cypher/neo4j
  • Nicer error messages
    • OnAttach fairing errors on incorrect database names
  • Docs

Things to do after this PR merges:

  • Rust MySQL TLS support
  • Rust Postgres TLS support
  • More adapters?
    • MongoDB
    • ODBC

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jebrosen jebrosen jebrosen requested changes
@SergioBenitez SergioBenitez SergioBenitez requested changes

Assignees

No one assigned

Labels

pr: merged This pull request was merged manually.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ELD @SergioBenitez @jebrosen