Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make flush of big number of dirty pages faster [CORE5660] #5926

Closed
firebird-automations opened this issue Nov 14, 2017 · 9 comments
Closed

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: @hvlad

Attachments:
sweep-3.0.2-hdd.png
sweep-3.0.2-ssd.png
sweep-3.0.3-hdd.png
sweep-3.0.3-ssd.png

Flush cache algorithm could be improved for the case when there is a big number of dirty pages to write.
Current algorithm adds noticeable CPU load and makes whole process slower than it could be.

Commits: d5f73f8 ebdf7b0

@firebird-automations
Copy link
Collaborator Author

Commented by: @hvlad

Graphs (by SysInternals Process Explorer) of sweep running by Firebird 3.0.2 Super Server on the 9GB TPCC database with page cache of 500K pages

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

Attachment: sweep-3.0.2-hdd.png [ 13184 ]

Attachment: sweep-3.0.2-ssd.png [ 13185 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

assignee: Vlad Khorsun [ hvlad ]

@firebird-automations
Copy link
Collaborator Author

Commented by: @hvlad

Same graphs with improved flush algorithm

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

Attachment: sweep-3.0.3-hdd.png [ 13186 ]

Attachment: sweep-3.0.3-ssd.png [ 13187 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

summary: Make flash of big number of dirty pages faster => Make flush of big number of dirty pages faster

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 3.0.3 [ 10810 ]

Fix Version: 4.0 Beta 1 [ 10750 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Resolved [ 5 ]

QA Status: No test => Cannot be tested

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Closed [ 6 ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment