dannycjones opened a new issue, #3229: URL: https://github.com/apache/iceberg-rust/issues/3229
### Apache Iceberg Rust version main ### Describe the bug Today, the data type parsing can parse some really weird combinations such as `decimal(5, 8))))))))))))))))))))))))))))))))))))))))))))))))))))))))))))))))`. This is not a big issue, but I think it would make more sense for us to parse a stricter subset - not overly strict, but one that is predictable. ### To Reproduce _No response_ ### Expected behavior We should reject types that contain additional `)` or otherwise appears malformed. I'd recommend we adopt a regex pattern, similar to iceberg-go. https://github.com/apache/iceberg-go/blob/8832cf696b5cc2a54767b5f2fa4c06b138c04495/types.go#L35-L40 ### Willingness to contribute I can contribute a fix for this bug independently -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
