[jira] [Updated] (HIVE-3474) get_partitions_ps is HORRIBLY broken

["Carl Steinbach (JIRA)" <ji...@apache.org>]

     [ 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