Hi Sergei, Please review a small patch that fixes the exact problem reported in 4634 (and nothing else). Thanks. (the other problems from the original big patch are now reported separately, with some of them already fixed). On 06/13/2013 12:08 PM, Sergei Golubchik wrote:
Hi, Alexander!
On Jun 13, Alexander Barkov wrote:
Hi, Alexander!
This looks like a lot of changes for what you explained the reason of the bug was. There will be lots of "whys" below...
The fix itself was quite small. However, an attempt to write a reasonable test case revealed a few other bugs. I just tried to fixing everything in a single patch.
I understood this and it was okay.
But I still wanted to know what change fixes what bug and what test case.
Okey. Let's do everything separately. These are separate reports for the problems found during writing a test case for 4634: https://mariadb.atlassian.net/browse/MDEV-4652 https://mariadb.atlassian.net/browse/MDEV-4653 https://mariadb.atlassian.net/browse/MDEV-4654
Here's a new one, found today: https://mariadb.atlassian.net/browse/MDEV-4651
Okay, as you prefer :)
Regards, Sergei