[FLINK-15538] Separate decimal implementations into separate sub-classes

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Invalid
• Component/s: Table SQL / Runtime
• Labels: auto-deprioritized-major pull-request-available
• Assignee: Unassigned
• Reporter: Liya Fan
• Created: 09/Jan/20 10:37
• Updated: 08/Oct/21 09:42
• Resolved: 09/Jun/21 07:59

Description

The current implementation of Decimal values have two (somewhat independent) implementations: one is based on Long, while the other is based on BigDecimal.

This makes the Decmial class not clear (both implementations cluttered in a single class) and less efficient (each method involves a if-else branch).

So in this issue, we make Decimal an abstract class, and separate the two implementation into two sub-classes. This makes the code clearer and more efficient.

Issue Links

• links to: GitHub Pull Request #10809