- 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 changedLines changed: 11 additions & 18 deletions
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)