[
https://issues.apache.org/jira/browse/FLINK-15538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17012419#comment-17012419
]
Jingsong Lee commented on FLINK-15538:
--------------------------------------
Bimorphic is slower than Monomorphic, and it will break the inline. If you take
a look to JDK {{BigDecimal}}, that also has {{long}} and {{BigInteger}} too. So
I suggestion you could provide some evidence to confirm efficient.
> Separate decimal implementations into separate sub-classes
> ----------------------------------------------------------
>
> Key: FLINK-15538
> URL: https://issues.apache.org/jira/browse/FLINK-15538
> Project: Flink
> Issue Type: Improvement
> Components: Table SQL / Runtime
> Reporter: Liya Fan
> Priority: Major
> Labels: pull-request-available
> Time Spent: 10m
> Remaining Estimate: 0h
>
> 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.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
