wecharyu opened a new pull request, #5246: URL: https://github.com/apache/hive/pull/5246
### What changes were proposed in this pull request? Fix the bug on `HMSHandler#checkLimitNumberOfPartitions`: 1. use maxParts to determine if need check partition limit 2. remove maxParts on `checkLimitNumberOfPartitions` ### Why are the changes needed? Fix the bug, assume that HMS configure `metastore.limit.partition.request` as 100, the client calls `get_partitions_by_filter` with maxParts as 101, and the matching partition number is 50, in this case the HMS server should not throw `MetaException` by partition limit check. ### Does this PR introduce _any_ user-facing change? No. ### Is the change a dependency upgrade? No. ### How was this patch tested? Fix some unit tests. -- 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]
