Hi, Sergey!

On Nov 01, Sergey Vojtovich wrote:
> revno: 3939
> revision-id: [email protected]
> parent: [email protected]
> committer: Sergey Vojtovich <[email protected]>
> branch nick: 5.5-mdev5205
> timestamp: Fri 2013-11-01 19:48:41 +0400
> message:
>   MDEV-5205 - MariaDB does not start if more than 128 cpu's are available
>   
> === modified file 'sql/threadpool_unix.cc'
> --- a/sql/threadpool_unix.cc  2013-06-22 12:02:03 +0000
> +++ b/sql/threadpool_unix.cc  2013-11-01 15:48:41 +0000
> @@ -907,6 +907,7 @@ int thread_group_init(thread_group_t *th
>    thread_group->pollfd= -1;
>    thread_group->shutdown_pipe[0]= -1;
>    thread_group->shutdown_pipe[1]= -1;
> +  thread_group->queue.empty();

ok, but please add to the definition of I_P_List::empty() a comment
that it's used in many places in the code instead of a constructor,
to initialize a bzero-ed I_P_List instance.

>    DBUG_RETURN(0);
>  }
>  
Regards,
Sergei

_______________________________________________
Mailing list: https://launchpad.net/~maria-developers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~maria-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to