Re: [Maria-developers] Please review a fix for MDEV-10780 (and likely for MDEV-10806 and MDEV-10910)

2016-11-10 Thread Oleksandr Byelkin
Hi, Alexander! On 10.11.2016 13:42, Alexander Barkov wrote: Hello Sanja, [skip] Here's an updated version with a comment and with a very small addition fixing another bugs. Patch is OK to push. (I'd removed "Start of 10.1 tests" becaus we mart normally ends only, but it is OK leave it as

Re: [Maria-developers] Please review a fix for MDEV-10780 (and likely for MDEV-10806 and MDEV-10910)

2016-11-10 Thread Alexander Barkov
Hello Sanja, On 11/10/2016 01:13 AM, Oleksandr Byelkin wrote: Hi, Alexander! On 09.11.2016 16:28, Alexander Barkov wrote: Hello Sanja, Elena, Wlad, Sanja, please review a patch for MDEV-10780. The patch is for 10.1. I could not reproduce it in 10.0. But it should be safe to push it into 10.

Re: [Maria-developers] Please review a fix for MDEV-10780 (and likely for MDEV-10806 and MDEV-10910)

2016-11-09 Thread Oleksandr Byelkin
Hi, Alexander! On 09.11.2016 16:28, Alexander Barkov wrote: Hello Sanja, Elena, Wlad, Sanja, please review a patch for MDEV-10780. The patch is for 10.1. I could not reproduce it in 10.0. But it should be safe to push it into 10.0 anyway. This patch is most likely fixing MDEV-10806 (assigned

Re: [Maria-developers] Please review a fix for MDEV-10780 (and likely for MDEV-10806 and MDEV-10910)

2016-11-09 Thread Elena Stepanova
Hi Alexander, On 11/09/2016 07:45 PM, Alexander Barkov wrote: Hello Elena, On 11/09/2016 08:38 PM, Elena Stepanova wrote: Hi Bar, Sanja, On 11/09/2016 06:28 PM, Alexander Barkov wrote: Hello Sanja, Elena, Wlad, Sanja, please review a patch for MDEV-10780. The patch is for 10.1. I could not

Re: [Maria-developers] Please review a fix for MDEV-10780 (and likely for MDEV-10806 and MDEV-10910)

2016-11-09 Thread Alexander Barkov
Hello Elena, On 11/09/2016 08:38 PM, Elena Stepanova wrote: Hi Bar, Sanja, On 11/09/2016 06:28 PM, Alexander Barkov wrote: Hello Sanja, Elena, Wlad, Sanja, please review a patch for MDEV-10780. The patch is for 10.1. I could not reproduce it in 10.0. But it should be safe to push it into 10.

Re: [Maria-developers] Please review a fix for MDEV-10780 (and likely for MDEV-10806 and MDEV-10910)

2016-11-09 Thread Elena Stepanova
Hi Bar, Sanja, On 11/09/2016 06:28 PM, Alexander Barkov wrote: Hello Sanja, Elena, Wlad, Sanja, please review a patch for MDEV-10780. The patch is for 10.1. I could not reproduce it in 10.0. But it should be safe to push it into 10.0 anyway. Yes, please push into 10.0 if it's safe enough, I

[Maria-developers] Please review a fix for MDEV-10780 (and likely for MDEV-10806 and MDEV-10910)

2016-11-09 Thread Alexander Barkov
Hello Sanja, Elena, Wlad, Sanja, please review a patch for MDEV-10780. The patch is for 10.1. I could not reproduce it in 10.0. But it should be safe to push it into 10.0 anyway. This patch is most likely fixing MDEV-10806 (assigned to Sanja) and MDEV-10910 (assigned to Elena). Thanks! diff --