[SERVER-68544] Make overflow decimal throws exception in classic engine Created: 03/Aug/22  Updated: 29/Oct/23  Resolved: 15/Sep/22

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: 6.2.0-rc0

Type: Bug Priority: Major - P3
Reporter: Zixuan Zhuang Assignee: Kevin Cherkauer
Resolution: Fixed Votes: 0
Labels: neweng
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
is depended on by SERVER-68679 Throw error on overflow for $add with... Closed
Related
is related to SERVER-67282 $add expression should not use Double... Closed
Backwards Compatibility: Fully Compatible
Operating System: ALL
Sprint: QE 2022-09-19
Participants:
Linked BF Score: 158

 Description   

The current behaviors of double and decimal are inconsistent:

  • Date + overflowDouble/nanDouble => overflow exception
  • Date + overflowDecimal/nanDecimal => nanDate

We can have decimal to throw exception as well for overflow: code link

 

(we will not backport this to 6.0 or earlier)

(this ticket is for classic engine, sbe change will be included in SERVER-67282)

 



 Comments   
Comment by Githook User [ 15/Sep/22 ]

Author:

{'name': 'Kevin Cherkauer', 'email': 'kevin.cherkauer@mongodb.com', 'username': 'kevin-cherkauer'}

Message: SERVER-68544 Assert on overflow of $add[Date, Decimal128] in Classic
Branch: master
https://github.com/mongodb/mongo/commit/43e1ac5b878b846c762cac228834088442371830

Comment by Brenda Rodriguez [ 09/Aug/22 ]

Saving this for Kevin.

Generated at Thu Feb 08 06:11:06 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.