[MDEV-11025] Make number of page cleaner threads variable dynamic Created: 2016-10-11 Updated: 2018-01-24 Resolved: 2017-10-24 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Storage Engine - InnoDB |
| Fix Version/s: | 10.3.3 |
| Type: | Task | Priority: | Major |
| Reporter: | Jan Lindström (Inactive) | Assignee: | Jan Lindström (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Description |
|
Currently number of page cleaner threads is static variable and requires server shutdown and restart to be able to change. |
| Comments |
| Comment by Jan Lindström (Inactive) [ 2017-09-01 ] |
|
https://github.com/MariaDB/server/commit/5ecaae034e9cc829685b8cefc2533b0d353d5c3f |
| Comment by Marko Mäkelä [ 2017-09-04 ] |
|
I think that making parameters dynamic is a good idea. |
| Comment by Jan Lindström (Inactive) [ 2017-09-05 ] |
|
https://github.com/MariaDB/server/commit/a3eaae42295499c242689aebfb9a5b001cdef614 |
| Comment by Jan Lindström (Inactive) [ 2017-09-05 ] |
|
Condition variables at the moment are used only to implement os_event_t, I do not want to make this patch bigger just by introducing a new OS-dependent object (yes, I could use/copy code from os_event_t). |
| Comment by Marko Mäkelä [ 2017-09-08 ] |
|
It seems that moving from events to condition variables would also require a move from InnoDB mutexes to native mutexes. OK, that would be a huge task on its own. The patch looks mostly OK, but it seems possible that it could introduce a hang on shutdown. |
| Comment by Jan Lindström (Inactive) [ 2017-10-24 ] |
|
commit b23a1096956c21df037bd851494f11509b5514dd Modified test cases New function buf_flush_set_page_cleaner_thread_cnt buf_flush_page_cleaner_worker New function innodb_page_cleaners_threads_update innobase_start_or_create_for_mysql |