sphinxsearch/sphinx_engine/sphinx.5.0.77.patch test failure

Bug #378879 reported by Neil Katin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OurDelta
Fix Committed
Medium
Unassigned

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.

Revision history for this message
Neil Katin (d-launchpad-askneil-com) wrote :
Changed in ourdelta:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Walter Heck (walterheck) wrote : Re: [Ourdelta-developers] [Bug 378879] Re: sphinxsearch/sphinx_engine/sphinx.5.0.77.patch test failure

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

Revision history for this message
Neil Katin (d-launchpad-askneil-com) wrote :

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

Changed in ourdelta:
status: Confirmed → Fix Committed
Revision history for this message
Arjen Lentz (arjen-lentz) wrote : Re: [Ourdelta-developers] [Bug 378879] Re: sphinxsearch/sphinx_engine/sphinx.5.0.77.patch test failure

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

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.