GIve error instead of warning if engine has been disabled

[Lee Bieber]

We currently only give a warning, this should be changed to give an error

#
# Test for handler type, will select MyISAM and print a warning
# about that - since NDB is disabled
#
create table t1 (id int) engine=NDB;
alter table t1 engine=NDB;
drop table t1;

[Lee Bieber (kalebral-deactivatedaccount)]

The unit test warnings_engine_disabled should be updated after this has been fixed.

[Jay Pipes (jaypipes)]

Confirmed. Assigning to low-hanging-fruit, as this should be fairly easy for someone to knock off.

[Padraig O'Sullivan (posulliv)]

I'd like to have a go fixing this as it sounds like an easy bug to get started with so I'm assigning it to myself if that's ok.

[Jay Pipes (jaypipes)]

Actually, it seems this may have been fixed already:

drizzle> use test
Database changed
drizzle> create table t1 (id int) engine=NDB;
ERROR 1286 (42000): Unknown table engine 'NDB'

So, I've fixed up the test case to reflect it...

Padraig, we'll have to attack another bug for you :)

[Jay Pipes (jaypipes)]

Fixed in ~jaypipes/drizzle/bug-fixes

[Lee Bieber (kalebral-deactivatedaccount)]

marking as fixed released as the fix seems to be in the trunk. But need to update the test case - warnings_engine_disabled