Skip to content

Conversation

@morrySnow
Copy link
Contributor

@morrySnow morrySnow commented Nov 23, 2023

Proposed changes

Issue Number: close #xxx

  1. decimalv3: use 38, min(decimalOverflowScale, ret_scale) as result precision if overflow
  2. decimalv2 with largeint or bigint will promotion to decimalv3
  3. decimalOverflowScale could be config by session variable

TODO:

  1. update legacy planner?
  2. update be code to adjust this change

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@morrySnow morrySnow marked this pull request as draft November 23, 2023 11:26
@morrySnow morrySnow force-pushed the decimal_new branch 5 times, most recently from 18d1b58 to b1b30f9 Compare November 24, 2023 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant