lp:~posulliv/drizzle/bug337038-dec-trunc-should-error

Branch merges

Propose for merging

No branches dependent on this one.

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

Jay Pipes (community): Approve on 2009-04-07

Diff: None lines api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #337038: DECIMAL truncation is warning, not error

High

Fix Released

Link a bug report

Related blueprints

975. By Padraig O'Sullivan on 2009-04-02

Added extra comments to parts of test cases that were modified for this bug
fix based on Jay's review comments. Also split multi-value INSERT statements
into single INSERT statements.

974. By Padraig O'Sullivan on 2009-04-01

Updating 1 more test result based on my fix.

973. By Padraig O'Sullivan on 2009-04-01

Fixed up 1 more test case based on the modifications I have made.

972. By Padraig O'Sullivan on 2009-04-01

Decided that we when we have an integer divided by an integer that returns a
truncated decimal result which will be stored in a temporary table, we will
error and store a default value in the temporary table when this occurs. The
other option was to have the query fail which would have required modifying
a lot of other things. I think this is a fair compromise as it is unlikely
someone is going to do something like divide (int) 1 / (int) 3 and expect to
get an accurate result. Most people would expect zero when dividing integers
like that.

971. By Padraig O'Sullivan on 2009-04-01

Fix for bug337038. Now we error on data truncation with decimal values but
the warning is still present so that a user can issue 'show warnings' and
still see the truncation warning.

970. By Brian Aker on 2009-03-30

Merge Monty.

969. By Brian Aker on 2009-03-30

Merge of Eric's libdrizzle work.

968. By Brian Aker on 2009-03-27

Merge from Brian

967. By Brian Aker on 2009-03-27

Merge Brian

966. By Brian Aker on 2009-03-26

Merge Stewart.