SEPURI-SAI-KRISHNA commented on issue #13022: URL: https://github.com/apache/seatunnel/issues/13022#issuecomment-6542983041
@SEZ9 thanks, both are in #13023 now: 1. `decimalKeepsItsPrecisionAndScale` asserts `decimal(76, 10)` maps to `DecimalType(76, 10)`, next to the `(10, 2)` and `(38, 0)` cases. I also added the capitalized `Decimal(10, 2)` form that older servers report. 2. The DECIMAL branch has a comment that precision and scale come from `COLUMN_SIZE` and `DECIMAL_DIGITS`, which the driver parses out of the full type name, not from the normalized name. One scope change, following a review on #13023: that PR now fixes only the DECIMAL part of this issue. BINARY is left as it is on `dev`, because the driver returns BINARY values as Strings, and declaring BYTES without converting the values would be misleading. That part stays open here for a follow-up. -- 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]
