jackylee-ch opened a new pull request, #956: URL: https://github.com/apache/paimon-rust/pull/956
`CoreOptions::bucket()` parses the `bucket` option with `unwrap_or(DEFAULT_BUCKET)`, so a bad value never surfaced: `bucket=0` was taken verbatim (silently turning off read-side bucket pruning), and a non-integer like `bucket=abc` silently fell back to dynamic bucketing (-1) instead of the fixed bucketing the user intended. This validates the option at create/alter time, mirroring Java `SchemaValidation.validateBucket`: -1 (dynamic), -2 (postpone) and any value >= 1 are allowed; 0, values <= -3, and non-integers are rejected with a clear error. (The Java `bucket=-1` + `bucket-key` arm is left to the existing bucket-key validation.) Tests cover the 0 / non-integer rejections and the accepted -1 / -2 / >=1 values. -- 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]
