zhilinli123 commented on PR #4634: URL: https://github.com/apache/incubator-seatunnel/pull/4634#issuecomment-1519297752
> From the design perspective, it is necessary to support any DecimalType with scale=0. Otherwise, DecimalType(20,0), DecimalType(30,0), etc., will need to be modified. So it's better to do it well in one go <img width="1422" alt="image" src="https://user-images.githubusercontent.com/76689593/233887127-3ae67dcb-7b2c-431d-bf83-2c09c4427c3a.png";> I have checked that Flink also uses Long type for storage. I think we do not need to change it for the time being CC @hailin0 @ic4y -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
![https://user-images.githubusercontent.com/76689593/233887127-3ae67dcb-7b2c-431d-bf83-2c09c4427c3a.png"](https://user-images.githubusercontent.com/76689593/233887127-3ae67dcb-7b2c-431d-bf83-2c09c4427c3a.png"){kind=link}