Bug #107715 Multiple SE clone not calling clone_end after mixed clone_begin failure/success

Description:
Suppose there are two clone-supporting storage engines, and that clone_begin succeeds in the first one, but fails in the second one. This results in clone_end not being called for the first storage engine, leaking the clone resources, and, in the case of InnoDB, hitting asserts in Clone_Sys destructor on shutdown.

Both remote and local clones are affected through different code paths. For remote clone, clone_hton.cc hton_clone_begin failing in 2nd or later SE will result in locator and task vectors containing only successful SE data - which is correct - but the caller will keep m_storage_initialized == false, skipping any hton_clone_end calls. For local clone, clone_hton.cc run_hton_clone_begin locator and task vectors are pre-populated, and a failure in the middle will result in them being half-initialized, with no indication which ones actually are initialized.

Note that clone_begin failing is not limited to actual errors happening during clone startup: in the case of clone using multiple threads some of them might be attempting to add tasks late, after the clone has already finished. Thus it is possible to hit this during regular successful clone.

How to repeat:
Code analysis

Suggested fix:
One option is to rollback the successfuly-initialized SEs in both hton_clone_begin and run_hton_clone_begin in the case of error.

Hello Laurynas,

Thank you for the report and feedback!

regards,
Umesh

Posted by developer:
 
Clone with Multiple SEs is not supported yet. The code present is to help future development.

Bug 107715 fix for 8.0.30

(*) I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.

Contribution: bug107715.patch (application/octet-stream, text), 2.17 KiB.

The patch applies to 8.0.31 too, MTR clone suite passes cleanly.

Applies cleanly on 8.0.32 and does not regress MTR

Applies cleanly on 8.0.33 and does not regress MTR

My contributed patch applies cleanly on 9.0.0 and passes the tests.

The contributed patch applies on 9.1.0 cleanly and passes the tests.