lp:~stewart/drizzle/jay-and-stewart-remove-pack_flag

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~drizzle-trunk/drizzle/development

Jay Pipes: Pending requested 2009-09-28

Brian Aker: Pending requested 2009-09-28

Monty Taylor: Pending requested 2009-09-28

Diff: 2218 lines

33 files modified

drizzled/common.h (+0/-1)
drizzled/create_field.cc (+10/-6)
drizzled/create_field.h (+1/-2)
drizzled/definitions.h (+4/-34)
drizzled/field.cc (+90/-56)
drizzled/field.h (+2/-1)
drizzled/field/blob.cc (+13/-19)
drizzled/field/blob.h (+36/-14)
drizzled/field/date.h (+21/-10)
drizzled/field/datetime.h (+21/-11)
drizzled/field/decimal.cc (+21/-8)
drizzled/field/decimal.h (+12/-5)
drizzled/field/enum.h (+13/-6)
drizzled/field/null.h (+68/-23)
drizzled/field/str.cc (+9/-5)
drizzled/field/str.h (+5/-3)
drizzled/field/timestamp.cc (+16/-11)
drizzled/field/timestamp.h (+10/-5)
drizzled/field/varstring.cc (+15/-9)
drizzled/field/varstring.h (+12/-7)
drizzled/function/func.cc (+7/-5)
drizzled/item.cc (+14/-6)
drizzled/item/insert_value.cc (+1/-2)
drizzled/item/sum.cc (+1/-2)
drizzled/item/type_holder.cc (+7/-3)
drizzled/key.cc (+1/-2)
drizzled/message/table.proto (+0/-2)
drizzled/sql_table.cc (+48/-71)
drizzled/statement/alter_table.cc (+1/-2)
drizzled/stored_key.h (+8/-3)
drizzled/table.cc (+218/-191)
drizzled/table_proto_write.cc (+32/-4)
plugin/innobase/handler/handler0alter.cc (+1/-2)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

1141. By Stewart Smith on 2009-09-28

fix up decimals type to be uint8_t throughout and check that value in the proto is valid for what the server can currently handle

1140. By Stewart Smith on 2009-09-24

remove pack_flag from table.proto

1139. By Stewart Smith on 2009-09-24

don't use pack_flag in make_field for creating Field_blob; instead use what we always encode it to (a DRIZZLE_TYPE_LONG) to calc_pack_length for creating the field. Currently have an assert for making sure this is correct. next patch, remove assert and storage of pack_flag in proto.

1138. By Stewart Smith on 2009-09-24

remove DECIMAL_FLAG, FIELDFLAG_DECIMAL_POSITION and f_is_decimal_precision all related to pack_flag and all effectively unused.

1137. By Stewart Smith on 2009-09-23

Fix maybe_null for CreateField created for temp table in item/sum.cc (Item_sum_distinct)
which was causing valgrind warning on linux x86 and sum_distinct test failures on
Solaris sparc.

1136. By Stewart Smith on 2009-09-22

merge with trunk

1135. By Stewart Smith on 2009-09-22

don't store decimal/double scale in pack_flag, instead use the numeric option scale field in the table proto. This removes f_decimals() macro and the bits in the pack_flag for scale.

1134. By Jay Pipes <jpipes@serialcoder> on 2009-08-26

Merge trunk and resolve conflicts.

1133. By Jay Pipes <jpipes@serialcoder> on 2009-08-26

Style cleanups after review. Thanks Monty :)

1132. By Jay Pipes <jpipes@serialcoder> on 2009-08-25

Merge trunk