fallintoplace opened a new pull request, #2474: URL: https://github.com/apache/iceberg-rust/pull/2474
### Summary - require exact `bucket[N]` and `truncate[W]` syntax when parsing transforms - reject zero values such as `bucket[0]` and `truncate[0]` - add parser coverage for valid, zero, and malformed parameterized transforms ### Why The previous parser stripped the transform prefix and trimmed brackets loosely, so malformed strings like `bucket10` and `truncate10` were accepted. It also accepted zero parameters, which can later panic when the transform is evaluated. Fixes #2473. ### Tests - `cargo fmt --check` - `cargo test -p iceberg parameterized_transform` - `cargo test -p iceberg spec::transform` - `cargo test -p iceberg bucket` - `cargo test -p iceberg truncate` -- 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]
