ic4y commented on PR #4634: URL: https://github.com/apache/incubator-seatunnel/pull/4634#issuecomment-1521087599
> > 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 alt="image" width="1422" 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 If Flink also supports bigint(20), it might have a similar issue. -- 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}