Make database connections configurable by environment variables #768

Samengevoegd

KitaitiMakoto heeft 4 commits samengevoegd van db-conns-conf naar master

KitaitiMakoto reageerde (Gemigreerd van github.com)

so that we can tune performance.

codecov[bot] reageerde (Gemigreerd van github.com)

Codecov Report

Merging #768 into master will decrease coverage by 0.03%.
The diff coverage is 18.18%.

@@            Coverage Diff             @@
##           master     #768      +/-   ##
==========================================
- Coverage   38.99%   38.96%   -0.04%     
==========================================
  Files          73       73              
  Lines        9721     9729       +8     
  Branches     2226     2227       +1     
==========================================
  Hits         3791     3791              
- Misses       4878     4885       +7     
- Partials     1052     1053       +1

# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/768?src=pr&el=h1) Report > Merging [#768](https://codecov.io/gh/Plume-org/Plume/pull/768?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/180e34b07c014a3a37b2031cf0532ee17104f47b&el=desc) will **decrease** coverage by `0.03%`. > The diff coverage is `18.18%`. ```diff @@ Coverage Diff @@ ## master #768 +/- ## ========================================== - Coverage 38.99% 38.96% -0.04% ========================================== Files 73 73 Lines 9721 9729 +8 Branches 2226 2227 +1 ========================================== Hits 3791 3791 - Misses 4878 4885 +7 - Partials 1052 1053 +1 ```

elegaanz (Gemigreerd van github.com) beoordeeld

elegaanz (Gemigreerd van github.com) heeft een reactie achtergelaten

Nice! I just have a small question, but otherwise it seems good to merge. Thanks!

plume-models/src/config.rs

elegaanz (Gemigreerd van github.com) reageerde

Is there a reason for panicking in one case and silently failing in the other?

KitaitiMakoto (Gemigreerd van github.com) beoordeeld

plume-models/src/config.rs

KitaitiMakoto (Gemigreerd van github.com) reageerde

Good catch! There's no reason why. I will rewrite this and push it.

KitaitiMakoto (Gemigreerd van github.com) beoordeeld

plume-models/src/config.rs

KitaitiMakoto (Gemigreerd van github.com) reageerde

Done.

elegaanz (Gemigreerd van github.com) heeft deze veranderingen goedgekeurd

elegaanz (Gemigreerd van github.com) heeft een reactie achtergelaten

Great!

KitaitiMakoto reageerde (Gemigreerd van github.com)

Thank you for the merge.

kiwii verwees naar dit probleem vanuit commit '

Make database connections configurable by environment variables (#768) * Make env DB_MAX_SIZE and DB_MIN_IDLE recognizable * Make database max size and minimum idle configurable * Restore file permission * Fail fast

Reviewers

De pull request is samengevoegd als 3be842c653.

Je kunt ook command line instructies bekijken.

Stap 1:

Vanuit het project, check een branch uit en test de veranderingen.

git checkout -b db-conns-conf master

git pull origin db-conns-conf

Stap 2:

Voeg de wijzigingen samen en update ze op Forgejo.

git checkout master

git merge --no-ff db-conns-conf

git push origin master

Geen beoordelaars

Geen mijlpaal

Niet toegewezen

1 deelnemers

Notificaties

Vervaldatum

De deadline is ongeldig of buiten bereik. Gebruik het formaat 'jjjj-mm-dd'.

Geen vervaldatum ingesteld.

Afhankelijkheden

Geen afhankelijkheden ingesteld.

Laden…

Er is nog geen inhoud.

Labels Mijlpalen

Make database connections configurable by environment variables #768

Codecov Report

Reviewers

Stap 1:

Stap 2: