jackylee-ch opened a new pull request, #10359: URL: https://github.com/apache/paimon/pull/10359
### Purpose `partition.expiration-max-num` is the maximum number of partitions to expire at a time (default `100`). `CoreOptions#partitionExpireMaxNum()` returned it unchecked, and `NormalPartitionExpire` caps the expired set with `.limit(min(size, maxExpireNum))`: `0` expires nothing (expired partitions and their files are never reclaimed) and a negative value makes `Stream#limit` throw on every expiration run — both silently, with no diagnostic. The adjacent `partition.expiration-batch-size` already degrades gracefully for non-positive values. This validates it in the getter with a clear message, matching the existing numeric-option checks. ### Tests `CoreOptionsTest#testPartitionExpireMaxNumRejectsNonPositive`: asserts `partitionExpireMaxNum()` throws `IllegalArgumentException` for `0` and `-1`. ### API and Format No change. ### Documentation No change. -- 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]
