sphinxsearch/sphinx_engine/sphinx.5.0.77.patch test failure

[Neil Katin]

ourdelta 5.0.77-d8 patch set fails the internal test suite when configured with the --with-debug=full
option.

The failure is an assertion error in sql/handler.cc:

This line appears in mysql-test/var/log/master.err:

mysqld: handler.cc:503: int ha_init(): Assertion `total_ha < 14' failed.

The underlying reason is the MAX_HA variable was not changed when the sphynx_engine was
added to the sys_table_types array in sql/handler.cc.

I'm not sure whether this error is benign or not. I think it may not be, because there are tables defined
to be of MAX_HA size that may not be null terminated any more.

The fix is to redefine MAX_HA from 14 to 15 as part of the patch. I'll push a branch to launchpad
with an updated patch with the fix.

[Neil Katin (d-launchpad-askneil-com)]

fix put in https://code.launchpad.net/~d-launchpad-askneil-com/ourdelta/ourdelta-fix-bug338879
and was proposed for integration.

[Walter Heck (walterheck)]

Hi,

On Wed, May 20, 2009 at 9:31 PM, Neil Katin <email address hidden> wrote:
> ** Branch linked: lp:~d-launchpad-askneil-com/ourdelta/ourdelta-fix-
> bug338879
>
You might want to change that branch name to have bug 378879 instead
of 338879 in it's name for sanity's sake?

Walter

[Neil Katin (d-launchpad-askneil-com)]

Sigh. Its always some step that gets messed up. Is there an easy way to change the branch name?
Thanks.

[Arjen Lentz (arjen-lentz)]

Hi Neil

On 21/05/2009, at 3:33 PM, Neil Katin wrote:
> Sigh. Its always some step that gets messed up. Is there an easy
> way to change the branch name?
> Thanks.

You can abandon this one (mark it as abandoned then it'll be gone),
then push another one.
Please let me know when that's done, looking at 5.0.82 build right now.

Cheers,
Arjen.

> --
> sphinxsearch/sphinx_engine/sphinx.5.0.77.patch test failure
> https://bugs.launchpad.net/bugs/378879
> You received this bug notification because you are a member of
> OurDelta-
> developers, which is the registrant for OurDelta.
>
> Status in OurDelta - Builds for MySQL: Confirmed
>
> Bug description:
>
> ourdelta 5.0.77-d8 patch set fails the internal test suite when
> configured with the --with-debug=full
> option.
>
> The failure is an assertion error in sql/handler.cc:
>
> This line appears in mysql-test/var/log/master.err:
>
> mysqld: handler.cc:503: int ha_init(): Assertion `total_ha < 14'
> failed.
>
> The underlying reason is the MAX_HA variable was not changed when
> the sphynx_engine was
> added to the sys_table_types array in sql/handler.cc.
>
> I'm not sure whether this error is benign or not. I think it may
> not be, because there are tables defined
> to be of MAX_HA size that may not be null terminated any more.
>
> The fix is to redefine MAX_HA from 14 to 15 as part of the patch.
> I'll push a branch to launchpad
> with an updated patch with the fix.
>
> _______________________________________________
> Mailing list: https://launchpad.net/~ourdelta-developers
> Post to : <email address hidden>
> Unsubscribe : https://launchpad.net/~ourdelta-developers
> More help : https://help.launchpad.net/ListHelp

--
Arjen Lentz, Director @ Open Query (http://openquery.com)
Affordable Training and ProActive Support for MySQL & related
technologies

Follow our blog at http://openquery.com/blog/
OurDelta: free enhanced builds for MySQL @ http://ourdelta.org