From: Zardosht Kasheff Date: September 29 2009 3:30pm Subject: Re: can show engine status ever return an error? List-Archive: http://lists.mysql.com/internals/37335 Message-Id: <2f9663ba0909290830w60dac7c8x6ba5e0b3ae456585@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Thanks Sergei, I filed the following bug: http://bugs.mysql.com/bug.php?id=3D47719 -Zardosht On Tue, Sep 29, 2009 at 11:26 AM, Sergei Golubchik wrote: > Hi, Zardosht! > > On Sep 29, Zardosht Kasheff wrote: >> >> >> In the function ha_show_status, in handler.cc, I see the following >> code to end the function: >> >> =A0 if (!result) >> =A0 =A0 my_eof(thd); >> =A0 return result; >> } >> >> So, if the handlerton's show_status function successfully returns, >> my_eof(thd) is called, and the assert is not hit in protocol.cc. >> However, if show_status returns an error, then nothing is called here, >> causing the assert. > > This looks like a bug to me. > >> Is there something simple and safe that can be done that allows >> show_status to return an error in some cases, but not causing a crash? > > The simple and safe workaround that you can do is to call my_error() > when you have an error. It will set the proper status in the > Diagnostics_area. > > Regards / Mit vielen Gr=FC=DFen, > Sergei > > -- > =A0 __ =A0___ =A0 =A0 ___ ____ =A0__ > =A0/ =A0|/ =A0/_ __/ __/ __ \/ / =A0 Sergei Golubchik > =A0/ /|_/ / // /\ \/ /_/ / /__ =A0Principal Software Engineer/Server Arch= itect > /_/ =A0/_/\_, /___/\___\_\___/ =A0Sun Microsystems GmbH, HRB M=FCnchen 16= 1028 > =A0 =A0 =A0 <___/ =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0Sonnenallee 1, 85551= Kirchheim-Heimstetten > Gesch=E4ftsf=FChrer: Thomas Schroeder, Wolfgang Engels, Wolf Frenkel > Vorsitzender des Aufsichtsrates: Martin H=E4ring >