lp:~stewart/drizzle/bug588251-startScan-unused-result
- Get this branch:
- bzr branch lp:~stewart/drizzle/bug588251-startScan-unused-result
Branch merges
- Drizzle Developers: Pending requested
-
Diff: 362 lines (+83/-40)13 files modifieddrizzled/cursor.cc (+6/-3)
drizzled/cursor.h (+2/-2)
drizzled/filesort.cc (+4/-2)
drizzled/item/subselect.cc (+6/-1)
drizzled/optimizer/quick_index_merge_select.cc (+5/-1)
drizzled/records.cc (+20/-9)
drizzled/records.h (+5/-5)
drizzled/sql_delete.cc (+7/-1)
drizzled/sql_select.cc (+8/-3)
drizzled/sql_update.cc (+10/-3)
drizzled/statement/alter_table.cc (+7/-1)
plugin/haildb/haildb_engine.cc (+3/-8)
plugin/haildb/haildb_engine.h (+0/-1)
Related bugs
Bug #588251: doStartTableScan() result not checked | Medium | Fix Released |
Related blueprints
Branch information
Recent revisions
- 2051. By Stewart Smith
-
remove the saving of previous_error in doStartTableScan in HailDB. Now that we've fixed the upper layer to handle errors returned from doStarttableScan correctly.
- 2050. By Stewart Smith
-
doStartTableScan() result not checked.
Add __attribute_
_((warn_ unused_ result) ) to Cursor: :startTableScan ()
and by extension init_read_recordsthen go and fix the places where we weren't checking for errors.
This patch helped by Monty Widenius' patch to MariaDB. Greatly increased
confidence that we either handle the errors correctly or are at least
on par with bugs :) - 2042. By Lee Bieber
-
Merge Andrew - fix bug 632755: Crash / assertion failed in drizzled:
:Session: :free_cached_ table (this=0x7f83a42 c63f0) at drizzled/ sql_base. cc:513
Merge Lee - update tests to run with --repeat=2
Merge Shrews - 695755: filtered_replicator test hangs after first run when run with --repeat=2
Branch metadata
- Branch format:
- Branch format 7
- Repository format:
- Bazaar repository format 2a (needs bzr 1.16 or later)
- Stacked on:
- lp:drizzle/7.0