[MDEV-27161] Add option for SQL thread to limit maximum execution time per query replicated Created: 2021-12-03 Updated: 2023-11-27 Resolved: 2022-08-03 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Replication |
| Fix Version/s: | 10.10.0 |
| Type: | Task | Priority: | Critical |
| Reporter: | Valerii Kravchuk | Assignee: | Brandon Nesterenko |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||
| Description |
|
Currently execution time for the replicated query is not limited in any clear way. On primary node we can use max_statement_time, but this value is neither passed to slave nor enforced for SQL thread when it's set locally. So, there is no way to "control" replication delay and just stop with error when it's too big already. I think option to stop with error after specific timeout (that may be different than local max_statement_time or primary's max_statement_time) would be useful for replication setups. |
| Comments |
| Comment by Andrei Elkin [ 2022-04-14 ] |
|
Slave appliers indeed ignore global.max_statement_time which they must be initialized with (as the user connection threads do). Whether that's designed on purpose (doubts) or is actually buggy remains to be seen. |
| Comment by Sergei Golubchik [ 2022-04-27 ] |
|
What do you mean "slave appliers indeed ignore global.max_statement_time which they must be initialized with (as the user connection threads do)" ? Every THD gets max_statement_time, slave thread too. It's copied from global in plugin_thdvar_init() which is called from THD::init(). What slave threads don't do — they don't set the query timer to expire after max_statement_time seconds. In THD::set_query_timer() slave threads are explicitly excluded. This looks very much like on purpose, one can hardly write slave_thread || unintentionally. |
| Comment by Sergei Golubchik [ 2022-04-27 ] |
|
I don't think it's a good idea. If the slave thread would always be using global.max_statement_time — that's one thing. But if you're adding an option anyway, better to have an option like --slave-max-statement-time and not --slave-uses-global-max-statement-time |
| Comment by Brandon Nesterenko [ 2022-07-05 ] |
|
Hi Andrei! This is ready for review. |
| Comment by Andrei Elkin [ 2022-08-03 ] |
|
Completed the work. The final review was provided by Sergei (kudos for pretty constructive notes and heads-ups!). |
| Comment by Michael Widenius [ 2023-05-19 ] |
|
The adopted solution was to add a new variable, Slave_max_statement_time, which can be |