Make database connections configurable by environment variables #768

Birleştirildi
KitaitiMakoto 2020-05-06 17:28:00 +00:00 db-conns-conf içindeki 4 işlemeyi master ile birleştirdi
KitaitiMakoto 2020-05-05 06:32:32 +00:00 yorum yaptı (github.com konumundan göç edildi)

so that we can tune performance.

so that we can tune performance.
codecov[bot] 2020-05-05 06:59:18 +00:00 yorum yaptı (github.com konumundan göç edildi)

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 (github.com konumundan göç edildi) 2020-05-05 21:45:21 +00:00 incelendi
elegaanz (github.com konumundan göç edildi) bir yorum yaptı

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

Nice! I just have a small question, but otherwise it seems good to merge. Thanks!
elegaanz (github.com konumundan göç edildi) 2020-05-05 21:43:59 +00:00 yorum yaptı

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

Is there a reason for panicking in one case and silently failing in the other?
KitaitiMakoto (github.com konumundan göç edildi) 2020-05-05 22:09:38 +00:00 incelendi
KitaitiMakoto (github.com konumundan göç edildi) 2020-05-05 22:09:38 +00:00 yorum yaptı

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

Good catch! There's no reason why. I will rewrite this and push it.
KitaitiMakoto (github.com konumundan göç edildi) 2020-05-05 22:11:41 +00:00 incelendi
KitaitiMakoto (github.com konumundan göç edildi) 2020-05-05 22:11:41 +00:00 yorum yaptı

Done.

Done.
elegaanz (github.com konumundan göç edildi) 2020-05-06 17:27:53 +00:00 bu değişiklikleri onayladı
elegaanz (github.com konumundan göç edildi) bir yorum yaptı

Great!

Great!
KitaitiMakoto 2020-05-06 20:01:59 +00:00 yorum yaptı (github.com konumundan göç edildi)

Thank you for the merge.

Thank you for the merge.
Bu konuşmaya katılmak için oturum aç.
Değerlendirici yok
Kilometre Taşı Yok
Proje yok
Atanan Kişi Yok
1 participant
Bildirimler
Bitiş Tarihi
Bitiş tarihi geçersiz veya aralık dışında. Lütfen 'yyyy-aa-gg' biçimini kullanın.

Bitiş tarihi atanmadı.

Bağımlılıklar

Bağımlılık yok.

Referans: Plume/Plume#768
Herhangi bir açıklama sağlanmadı.