[ 
https://issues.apache.org/jira/browse/HIVE-3474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carl Steinbach updated HIVE-3474:
---------------------------------

    Component/s: Thrift API
                 Metastore
    
> get_partitions_ps is HORRIBLY broken
> ------------------------------------
>
>                 Key: HIVE-3474
>                 URL: https://issues.apache.org/jira/browse/HIVE-3474
>             Project: Hive
>          Issue Type: Bug
>          Components: Metastore, Thrift API
>            Reporter: Jonathan Chang
>
> 1.) This function assumes that partial spec is in order of the partition 
> columns but it does no checking of this whatsoever.  In fact, it throws away 
> everything before the =.
> 2.) The regular expression constructed is incorrect because it lacks a 
> trailing '/'.  This means that asking for 'ds=2' will return 
> 'ds=2009-01-01/foo=bar'.
> 3.) There are multiple implementations of makePartName().  At the very least 
> a code cleanup pass should happen.  There is a potentially subtle problem 
> with the one in Warehouse that takes a Map as input since it may not preserve 
> the partition order (which is critical since it is just doing regexp 
> checking).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to