Docs restructure #2953
Docs restructure #2953
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #2953 +/- ##
==========================================
- Coverage 66.49% 66.11% -0.39%
==========================================
Files 499 499
Lines 53790 53799 +9
==========================================
- Hits 35770 35569 -201
- Misses 14484 14636 +152
- Partials 3536 3594 +58
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
| defer close() | ||
| defer func() { | ||
| // give components the time to process purge requests | ||
| time.Sleep(time.Millisecond * 50) |
There was a problem hiding this comment.
Seems like this will still be quite flakey. Is there a better way to handle this in the test?
|
|
||
| ```yaml | ||
| global: | ||
| # ... | ||
| database: | ||
| connection_string: postgres://user:pass@hostname/database?sslmode=disable | ||
| max_open_conns: 100 | ||
| max_open_conns: 90 |
There was a problem hiding this comment.
out of curiosity, why the change in number here? :)
There was a problem hiding this comment.
As mentioned, the default configuration for Postgres is set to 100, but has reserved 3 slots for administrative usage. So having 90 conns for Dendrite sounds like a fair trade. (Even though we probably could go further down)
Needs to be merged into
gh-pageslater on.