- Notifications
You must be signed in to change notification settings - Fork5
Commit5364b35
committed
Increase maximum number of clog buffers.
Benchmarking has shown that the current number of clog buffers limitsscalability. We've previously increased the number in33aaa13, butthat's not sufficient with a large number of clients.We've benchmarked the cost of increasing the limit by benchmarking worstcase scenarios; testing showed that 128 buffers don't cause aregression, even in contrived scenarios, whereas 256 doesThere are a number of more complex patches flying around to addressvarious clog scalability problems, but this is simple enough that we canget it into 9.6; and is beneficial even after those patches have beenapplied.It is a bit unsatisfactory to increase this in small steps every fewreleases, but a better solution seems to require a rewrite of slru.c;not something done quickly.Author: Amit Kapila and Andres FreundDiscussion: CAA4eK1+-=18HOrdqtLXqOMwZDbC_15WTyHiFruz7BvVArZPaAw@mail.gmail.com1 parent25fe8b5 commit5364b35
1 file changed
+11
-18
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
417 | 417 | | |
418 | 418 | | |
419 | 419 | | |
420 | | - | |
421 | | - | |
422 | | - | |
423 | | - | |
424 | | - | |
425 | | - | |
| 420 | + | |
| 421 | + | |
| 422 | + | |
| 423 | + | |
| 424 | + | |
426 | 425 | | |
427 | | - | |
428 | | - | |
429 | | - | |
430 | | - | |
| 426 | + | |
| 427 | + | |
| 428 | + | |
| 429 | + | |
431 | 430 | | |
432 | | - | |
433 | | - | |
434 | | - | |
435 | | - | |
436 | | - | |
437 | | - | |
438 | | - | |
| 431 | + | |
439 | 432 | | |
440 | 433 | | |
441 | 434 | | |
442 | 435 | | |
443 | | - | |
| 436 | + | |
444 | 437 | | |
445 | 438 | | |
446 | 439 | | |
| |||
0 commit comments
Comments
(0)