[
https://issues.apache.org/jira/browse/HIVE-18567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16353809#comment-16353809
]
Adam Szita commented on HIVE-18567:
-----------------------------------
[~spena] I see. Although in PartitionIterable class this method is called with
(short)-1, so it would still give back all partition names, let's not change
that bit then (also for possible backward incompatibility reasons). Instead I
propose making the description of this conf var more accurate as per
[^HIVE-18567.1.patch].
Let me know what you think.
> ObjectStore.getPartitionNamesNoTxn doesn't handle max param properly
> --------------------------------------------------------------------
>
> Key: HIVE-18567
> URL: https://issues.apache.org/jira/browse/HIVE-18567
> Project: Hive
> Issue Type: Bug
> Components: Metastore
> Reporter: Adam Szita
> Assignee: Adam Szita
> Priority: Major
> Attachments: HIVE-18567.0.patch, HIVE-18567.1.patch
>
>
> As per [this HMS API test
> case|https://github.com/apache/hive/commit/fa0a8d27d4149cc5cc2dbb49d8eb6b03f46bc279#diff-25c67d898000b53e623a6df9221aad5dR1044]
> listing partition names doesn't check tha max param against
> MetaStoreConf.LIMIT_PARTITION_REQUEST (as other methods do by
> checkLimitNumberOfPartitionsByFilter), and also behaves differently on max=0
> setting compared to other methods.
> We should bring this into consistency.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
