[MDEV-25336] Parallel replication causes failed assert while restarting Created: 2021-04-05 Updated: 2021-05-18 Resolved: 2021-05-14 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Replication |
| Affects Version/s: | 10.2, 10.3 |
| Fix Version/s: | 10.2.39, 10.3.30, 10.4.20, 10.5.11 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Sachin Setiya (Inactive) | Assignee: | Sachin Setiya (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Environment: |
Arch Linux, Ubuntu |
||
| Description |
|
Test Case
It does not fail in 10.5 Failure
|
| Comments |
| Comment by Sachin Setiya (Inactive) [ 2021-04-05 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||
|
With rr it gives different call stack
| |||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Sachin Setiya (Inactive) [ 2021-04-06 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||
|
So the issue is this We have this kill server thread which calls close_connections()
when we call threads.get() , It unlinks the elements from link link as it returns
But in handle_rpl_parallel_thread, when we call
So if threads.get() in close connection is called before worker threads gets time to clean up we will get this assert failure | |||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Sachin Setiya (Inactive) [ 2021-04-13 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||
|
It does not fail in 10.1 , In 10.1 we use
instead of assert on unlinking | |||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Sachin Setiya (Inactive) [ 2021-04-14 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||
|
Need to backport | |||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Sachin Setiya (Inactive) [ 2021-04-14 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||
|
Patch branch bb-10.2-sachin | |||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Andrei Elkin [ 2021-04-21 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||
|
Asked questions, suggested todo:s. | |||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Sachin Setiya (Inactive) [ 2021-04-29 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||
|
Patch updated bb-10.2-sachin | |||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Andrei Elkin [ 2021-05-07 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||
|
The patch looks good! Thanks. |