[CALCITE-6324] Type inferred for result of STDDEV, VAR_SAMP, etc. is incorrect

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Invalid
• Affects Version/s: 1.36.0
• Fix Version/s: 1.37.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Mihai Budiu
• Reporter: Mihai Budiu
• Created: 12/Mar/24 23:21
• Updated: 22/Apr/24 18:27
• Resolved: 13/Mar/24 20:09

Description

These functions are all use the same type inference algorithm, essentially the algorithm used by AVG.

But if the values processed are decimal, STDDEV (and others) need much higher precision to represent the result. (I am not sure that the inference is right for integer types either, btw.)

This surfaced during the implementation of a fix for CALCITE-6322: if we use the type inferred for these functions, the result overflows and causes a runtime exception.

Issue Links

• is related to: CALCITE-6322 Casts to DECIMAL types are ignored
• links to: GitHub Pull Request #3731