lp:~stewart/drizzle/embedded-innodb-type-decimal

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~drizzle-trunk/drizzle/development at revision 1572

Drizzle Developers: Pending requested 2010-05-27

Brian Aker: Pending requested 2010-05-27

Diff: 1260 lines (+1220/-0)

7 files modified

plugin/embedded_innodb/embedded_innodb_engine.cc (+4/-0)
plugin/embedded_innodb/test-suite-dir/embedded_innodb/tests/r/basic_type_decimal.result (+8/-0)
plugin/embedded_innodb/test-suite-dir/embedded_innodb/tests/r/type_decimal.result (+599/-0)
plugin/embedded_innodb/test-suite-dir/embedded_innodb/tests/t/basic_type_decimal-master.opt (+1/-0)
plugin/embedded_innodb/test-suite-dir/embedded_innodb/tests/t/basic_type_decimal.test (+4/-0)
plugin/embedded_innodb/test-suite-dir/embedded_innodb/tests/t/type_decimal-master.opt (+1/-0)
plugin/embedded_innodb/test-suite-dir/embedded_innodb/tests/t/type_decimal.test (+603/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

1490. By Stewart Smith on 2010-05-27

Merged embedded-innodb-type-blob into embedded-innodb-type-decimal.

1489. By Stewart Smith on 2010-05-27

Merged embedded-innodb-type-blob into embedded-innodb-type-decimal.

1488. By Stewart Smith on 2010-05-27

add type_decimal test for embedded_innodb

1487. By Stewart Smith on 2010-05-27

very basic decimal type test for embedded_innodb

1486. By Stewart Smith on 2010-05-27

very simple addition to create fields of DECIMAL type

1485. By Stewart Smith on 2010-05-27

we actually don't need to check the result, just set status correctly

1484. By Stewart Smith on 2010-05-26

add a copy of type_blob for embedded_innodb

1483. By Stewart Smith on 2010-05-26

check that we really did move to the right row in EmbeddedInnoDBCursor::innodb_index_read(). This emans type_blob test now passes. We need to update table->status as well here.

1482. By Stewart Smith on 2010-05-26

doStartTableScan() can be called twice in a row without a doEndTableScan()

Showing up in count_distinct test with embedded_innodb.

Inspection of innobase handler shows that this would be okay for innobase.

What the fuck is the point of a programming interface if all the rules of how you are called are violated?

1481. By Stewart Smith on 2010-05-26

the memory management for BLOBS is completely insane. For certain types of index lookups we *must* do a copy and clear it *only* in ::reset() or ::extra(). This patch fixes up any valgrind warnings/assertions for embedded_innodb executing type_blob test.