Updates in performance.rst #8410
Conversation
|
Thank you for your first contribution @Erdou. You are absolutely right with the changes you propose. However, I am afraid that this greatly overlaps with the ongoing work from #7960. Maybe you would like to merge your changes with the changes done by @javiereguiluz? @javiereguiluz Or should we merge here and you rebase your changes on top of that? |
|
Thank you for your feedback @xabbuh |
|
@xabbuh @javiereguiluz Merge done and I also improved a few minor stuff, but Travis is not happy and I cannot understand why. A little help will be appreciated here :) |
|
@Erdou the error comes from this change:
The |
|
@javiereguiluz Thanks it passes now :) |
|
@javiereguiluz Can you have a look here? |
This PR was merged into the 3.3 branch. Discussion ---------- Remove outdated performance tips I've just realized there is an ongoing effort to rewrite this in #8410 and #7960. But if those are not merged soon, I'd suggest to merge this first so that we can at least solve the broken doc and people don't try to repeatedly fix this, like me and #8332 Commits ------- d21e200 Remove outdated performance tips
|
Closing it in favor of #9161. |
…onable (javiereguiluz) This PR was squashed before being merged into the 2.7 branch (closes #9161). Discussion ---------- Updated the main performance article to make it more actionable This one finishes and replaces #7960, #8410 and #8332. Commits ------- fa393c5 Updated the main performance article to make it more actionable
…with generated APCu loader