babelfish-for-postgresql / babelfish_extensions / 14261030212

74%

BABEL_5_X_DEV: 77%

Build Type

push

github

Committed by web-flow

Commit Message

Fix issue with truncation of result of arithmetic computations to correct scale when result is of numeric datatype (#3648)

* Fix issue with truncation of result of arithmetic computations to correct scale when result is of numeric datatype (#3455)

Currently, whenever an arithmetic operation is performed which results in numeric/decimal, we are storing the result till maximum possible scale instead of the correct scale. This is causing incorrect results in some arithmetic operations such as numeric division, numeric multiplication etc. This PR will fix this issue by adding a hooks in engine which truncates/rounds the result value to the correct scale and also sets the correct typmod of multiple nodes whose base type is NUMERIC.

This PR also fixes precision overflow adjustment logic in case NUMERIC ADDITION, SUBTRACTION and MULTIPLICATION.

Engine PR: babelfish-for-postgresql/postgresql_modified_for_babelfish#529

Issues Resolved
BABEL-5467, BABEL-5685

Signed-off-by: Rohit Bhagat <rohitbgt@amazon.com>

Run Details

148 of 163 new or added lines in 3 files covered. (90.8%)

3 existing lines in 2 files now uncovered.

43733 of 59086 relevant lines covered (74.02%)

177781.67 hits per line