Re: [Maria-developers] 099596296ce: Avoid mallocs when using LONGLONG_BUFFER_SIZE

2020-09-11 Thread Sergei Golubchik
Hi, Michael! This is a good catch! But note that LONGLONG_BUFFER_SIZE already reserves +1 for '\0', it's defined as /* Max length needed for a buffer to hold a longlong or ulonglong + end \0 */ #define LONGLONG_BUFFER_SIZE 21 The problem is that String::set_int doesn't use

Re: [Maria-developers] 2de0c74f94c: Optimize Sql_alloc

2020-09-11 Thread Sergei Golubchik
Hi, Michael! ok to push On Sep 08, Michael Widenius wrote: > revision-id: 2de0c74f94c (mariadb-10.5.2-271-g2de0c74f94c) > parent(s): 21eb8969ce9 > author: Michael Widenius > committer: Michael Widenius > timestamp: 2020-09-03 15:17:56 +0300 > message: > > Optimize Sql_alloc > > - Remove

Re: [Maria-developers] 21eb8969ce9: Improved storage size for Item, Field and some other classes

2020-09-11 Thread Sergei Golubchik
Hi, Monty! Looks ok, but again it doesn't seem you've squashed intermediate commits as you said you did. Bit fields and non-existent commits in columnstore - it's is clearly an intermediate work-in-progress state, all fixed in your later commits. On Sep 08, Michael Widenius wrote: >

Re: [Maria-developers] 23814b75241: Report memory leaks from mariadbd if -T or --debug is used

2020-09-11 Thread Sergei Golubchik
Hi, Michael! ok to push, of course On Sep 08, Michael Widenius wrote: > revision-id: 23814b75241 (mariadb-10.5.2-273-g23814b75241) > parent(s): 099596296ce > author: Michael Widenius > committer: Michael Widenius > timestamp: 2020-09-03 15:17:56 +0300 > message: > > Report memory leaks from