Merge lp://qastaging/~stewart/drizzle/misc-cleanups into lp://qastaging/drizzle

Proposed by Stewart Smith
Status: Merged
Merged at revision: 2631
Proposed branch: lp://qastaging/~stewart/drizzle/misc-cleanups
Merge into: lp://qastaging/drizzle
Diff against target: 938 lines (+169/-139)
43 files modified
drizzled/base.h (+0/-8)
drizzled/cursor.cc (+1/-0)
drizzled/cursor.h (+2/-2)
drizzled/field.h (+1/-1)
drizzled/ha_commands.cc (+0/-5)
drizzled/handler_structs.h (+4/-3)
drizzled/include.am (+2/-1)
drizzled/item/field.cc (+1/-0)
drizzled/item/subselect.cc (+1/-0)
drizzled/join.cc (+1/-0)
drizzled/key.cc (+1/-0)
drizzled/key.h (+2/-0)
drizzled/key_info.h (+5/-63)
drizzled/key_part_info.h (+61/-0)
drizzled/optimizer/key_field.cc (+1/-0)
drizzled/optimizer/quick_group_min_max_select.cc (+1/-0)
drizzled/optimizer/quick_group_min_max_select.h (+2/-0)
drizzled/optimizer/quick_range_select.h (+3/-0)
drizzled/optimizer/range.cc (+4/-2)
drizzled/optimizer/sum.cc (+1/-0)
drizzled/optimizer/table_read_plan.h (+3/-0)
drizzled/plugin/storage_engine.h (+1/-1)
drizzled/sql_select.cc (+1/-0)
drizzled/sql_select.h (+2/-0)
drizzled/sql_table.cc (+1/-0)
drizzled/sql_yacc.yy (+3/-3)
drizzled/statement/alter_table.cc (+1/-0)
drizzled/table.cc (+3/-2)
drizzled/table.h (+29/-2)
drizzled/table/instance/base.cc (+3/-18)
drizzled/table/instance/base.h (+1/-0)
drizzled/table/singular.cc (+1/-0)
drizzled/table_proto.cc (+6/-16)
plugin/innobase/handler/ha_innodb.cc (+2/-1)
plugin/innobase/handler/handler0alter.cc (+1/-0)
plugin/memory/ha_heap.cc (+2/-1)
plugin/myisam/ha_myisam.cc (+4/-3)
plugin/myisam/mi_range.cc (+2/-1)
plugin/myisam/mi_rkey.cc (+2/-1)
plugin/myisam/mi_rnext.cc (+3/-2)
plugin/myisam/mi_rnext_same.cc (+2/-1)
plugin/storage_engine_api_tester/storage_engine_api_tester.cc (+1/-1)
plugin/tableprototester/tableprototester.cc (+1/-1)
To merge this branch: bzr merge lp://qastaging/~stewart/drizzle/misc-cleanups
Reviewer Review Type Date Requested Status
Stewart Smith (community) Approve
Drizzle Trunk Pending
Review via email: mp+151404@code.qastaging.launchpad.net

Description of the change

a bunch of misc cleanups, even getting rid of the legacy structs.h

To post a comment you must log in.
Revision history for this message
Stewart Smith (stewart) wrote :

Approved by Brian over mail.

review: Approve
2637. By Stewart Smith

fix a condition in ha_myisam::doOpen() that was relying on message::Table::STANDARD being 0 and non-standard tables being non-zero. At least let's not rely on message::Table::STANDARD being 0.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: