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

Use Pool instead of Client from postgres module #77

Open
3 tasks
Narigo opened this issue Jun 14, 2020 · 0 comments
Open
3 tasks

Use Pool instead of Client from postgres module #77

Narigo opened this issue Jun 14, 2020 · 0 comments
Labels
⚙️ backend Related to the backend (NodeJS, Docker, Build) ⚡ new New tasks that were recently added and need attention from project management
Milestone

Comments

@Narigo
Copy link
Contributor

Narigo commented Jun 14, 2020

Description
The backend uses a single Client for all queries to the database in a single shop. It could use the Pool and work with just a couple of connections for all shops of an instance.

Links/References
(Add any designs, screenshots or links that helpful to implement the feature)

Tasks

  • Check that each shop still has its own prefix "connection"
  • Use the same pool for the real database connection
  • Test that everything still works with multiple connections

Additional Context
The current architecture should work for a couple of shops if there are not too many people reserving slots at the same time. Using a Pool instead would make it scale better if you decide to host multiple highly frequented shops.

@Narigo Narigo added the ⚙️ backend Related to the backend (NodeJS, Docker, Build) label Jun 14, 2020
@Narigo Narigo added this to the Version 1 milestone Jun 14, 2020
@Narigo Narigo added the ⚡ new New tasks that were recently added and need attention from project management label Jun 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ backend Related to the backend (NodeJS, Docker, Build) ⚡ new New tasks that were recently added and need attention from project management
Projects
None yet
Development

No branches or pull requests

1 participant