You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by "Namit Jain (Created) (JIRA)" <ji...@apache.org> on 2012/02/10 21:14:59 UTC

[jira] [Created] (HIVE-2799) change the following thrift apis to add a region

change the following thrift apis to add a region
------------------------------------------------

                 Key: HIVE-2799
                 URL: https://issues.apache.org/jira/browse/HIVE-2799
             Project: Hive
          Issue Type: New Feature
            Reporter: Namit Jain
            Assignee: Namit Jain


 list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
  list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
  Table get_table(1:string dbname, 2:string tbl_name)
                       throws (1:MetaException o1, 2:NoSuchObjectException o2)
  list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
           throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
  list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
                       throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
  Partition add_partition(1:Partition new_part)
                       throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
  i32 add_partitions(1:list<Partition> new_parts)
                       throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
  Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
                       throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
  Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
                       throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
  bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
                       throws(1:NoSuchObjectException o1, 2:MetaException o2)
  bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
                       throws(1:NoSuchObjectException o1, 2:MetaException o2)
  Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
                       throws(1:MetaException o1, 2:NoSuchObjectException o2)

  Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
      4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)

  Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
                       throws(1:MetaException o1, 2:NoSuchObjectException o2)
  list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
                       throws(1:NoSuchObjectException o1, 2:MetaException o2)
  list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
     4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)

  list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
                       throws(1:MetaException o2)
  list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
    3:list<string> part_vals, 4:i16 max_parts=-1)
                       throws(1:MetaException o1, 2:NoSuchObjectException o2)
  list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
     5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)

  list<string> get_partition_names_ps(1:string db_name,
    2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
                       throws(1:MetaException o1, 2:NoSuchObjectException o2)
  list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
    3:string filter, 4:i16 max_parts=-1)
                       throws(1:MetaException o1, 2:NoSuchObjectException o2)
  list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
                       throws(1:MetaException o1, 2:NoSuchObjectException o2)
  bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
                       throws(1:NoSuchObjectException o1, 2:MetaException o2)
  Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
                       throws(1:MetaException o1, 2:NoSuchObjectException o2)

  list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
                       throws(1:NoSuchObjectException o1, 2:MetaException o2)
  list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
                       throws(1:MetaException o2)


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HIVE-2799) change the following thrift apis to add a region

Posted by "Ashutosh Chauhan (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13207134#comment-13207134 ] 

Ashutosh Chauhan commented on HIVE-2799:
----------------------------------------

I assume all of these apis will be backward compatible because thrift rpc will take care of it. Namit/Kevin can you please confirm?
                
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HIVE-2799) change the following thrift apis to add a region

Posted by "Kevin Wilfong (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13207033#comment-13207033 ] 

Kevin Wilfong commented on HIVE-2799:
-------------------------------------

Adding the following to the list to be changed.

void create_table(1:Table tbl) throws(1:AlreadyExistsException o1, 2:InvalidObjectException o2, 3:MetaException o3, 4:NoSuchObjectException o4)
void drop_table(1:string dbname, 2:string name, 3:bool deleteData)
                       throws(1:NoSuchObjectException o1, 2:MetaException o3)
Index add_index(1:Index new_index, 2: Table index_table)
                       throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
                
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HIVE-2799) change the following thrift apis to add a region

Posted by "Kevin Wilfong (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13207206#comment-13207206 ] 

Kevin Wilfong commented on HIVE-2799:
-------------------------------------

Thanks, Carl, that sounds like a good idea.

Unfortunately, Thrift doesn't support method overloading so I'll still have to create new methods with new names to provide backwards compatibility.  This should make it easier to make changes like this in the future though.
                
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore, Thrift API
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HIVE-2799) change the following thrift apis to add a region

Posted by "Kevin Wilfong (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13209016#comment-13209016 ] 

Kevin Wilfong commented on HIVE-2799:
-------------------------------------

I'm definitely in favor of that approach.  The primary reason I intended to add duplicate Thrift API calls was to keep open source users happy, but if people are content with simply wrapping them in HiveMetaStoreClient, I am more than happy to oblige.
                
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore, Thrift API
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HIVE-2799) change the following thrift apis to add a region

Posted by "Carl Steinbach (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Carl Steinbach updated HIVE-2799:
---------------------------------

    Component/s: Thrift API
                 Metastore
    
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore, Thrift API
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HIVE-2799) change the following thrift apis to add a region

Posted by "Kevin Wilfong (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13208972#comment-13208972 ] 

Kevin Wilfong commented on HIVE-2799:
-------------------------------------

I was under the impression that that's true in the sense that, if you are using an old client which thinks a method takes 1 parameter and the server is running new code which thinks that method takes 2 parameters it will still work.  However, once you upgrade your client to use new code, you will always have to provide 2 parameters, at least if the client is in Java, I'm not sure if this applies to all languages Thrift supports.

I wanted to makes sure that this would not break code that uses the current Thrift APIs, even after the clients are upgraded.
                
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore, Thrift API
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HIVE-2799) change the following thrift apis to add a region

Posted by "Kevin Wilfong (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13207144#comment-13207144 ] 

Kevin Wilfong commented on HIVE-2799:
-------------------------------------

I will leave the current API's unchanged, I intend to add additional APIs, like create_table_using_region, which will allow the user to provide a region name in addition to all normal arguments.
                
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HIVE-2799) change the following thrift apis to add a region

Posted by "Carl Steinbach (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13207193#comment-13207193 ] 

Carl Steinbach commented on HIVE-2799:
--------------------------------------

I recommend adding methods that take a single "request" object as input, e.g:

void create_table(1: CreateTableReq req) throws (1:AlreadyExistsException o1, 2:InvalidObjectException o2, 3:MetaException o3, 4:NoSuchObjectException o4)

My understanding of Thrift is that new properties can be added to the CreateTableReq object without breaking backwards compatibility.
                
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HIVE-2799) change the following thrift apis to add a region

Posted by "Ashutosh Chauhan (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13208988#comment-13208988 ] 

Ashutosh Chauhan commented on HIVE-2799:
----------------------------------------

In case client is upgraded but server is not, my impression is that extra param passed on by client is automatically dropped by rpc before making a call on server side. So, that will still work. 
                
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore, Thrift API
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Assigned] (HIVE-2799) change the following thrift apis to add a region

Posted by "Namit Jain (Assigned) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Namit Jain reassigned HIVE-2799:
--------------------------------

    Assignee: Kevin Wilfong  (was: Namit Jain)
    
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HIVE-2799) change the following thrift apis to add a region

Posted by "Ashutosh Chauhan (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13208945#comment-13208945 ] 

Ashutosh Chauhan commented on HIVE-2799:
----------------------------------------

My understanding of thrift is limited. But, AFAIK you can add new params into existing thrift api and they will still be backward compatible. Is that not the case? Or, is there any other reason that you want to add a whole new set of parallel apis?
                
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore, Thrift API
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HIVE-2799) change the following thrift apis to add a region

Posted by "Ashutosh Chauhan (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13209013#comment-13209013 ] 

Ashutosh Chauhan commented on HIVE-2799:
----------------------------------------

For that we can modify HiveMetaStoreClient.java (the most widely used client) to wrap these methods in the one which don't take region as an argument (which is the current api) and then passing null for server param through rpc client. Those folks who are using real rpc clients their clients will continue to work without recompilation and if they are indeed recompiling they can pass on a null in there. 

At this point, I think we should reconsider whether we want to add a new set of apis or want to modify the existing ones. To me, latter seems a better choice to avoid code duplication and confusion.
                
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore, Thrift API
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HIVE-2799) change the following thrift apis to add a region

Posted by "Kevin Wilfong (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13208997#comment-13208997 ] 

Kevin Wilfong commented on HIVE-2799:
-------------------------------------

@Ashutosh

I agree, that will work as well.  I meant I wanted to make sure I don't force people who don't want to use multi-region to have to add a region to their Thrift API calls, once both the server and client are upgraded.
                
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore, Thrift API
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Resolved] (HIVE-2799) change the following thrift apis to add a region

Posted by "Kevin Wilfong (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kevin Wilfong resolved HIVE-2799.
---------------------------------

    Resolution: Not A Problem
    
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore, Thrift API
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HIVE-2799) change the following thrift apis to add a region

Posted by "Ashutosh Chauhan (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13209485#comment-13209485 ] 

Ashutosh Chauhan commented on HIVE-2799:
----------------------------------------

As a developer of Hive, I am definitely in favor of keeping code simpler and interfaces cleaner. As an open source user, I am fine with wrapping the new calls in existing apis.  
                
> change the following thrift apis to add a region
> ------------------------------------------------
>
>                 Key: HIVE-2799
>                 URL: https://issues.apache.org/jira/browse/HIVE-2799
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore, Thrift API
>            Reporter: Namit Jain
>            Assignee: Kevin Wilfong
>
>  list<string> get_tables(1: string db_name, 2: string pattern) throws (1: MetaException o1)
>   list<string> get_all_tables(1: string db_name) throws (1: MetaException o1)
>   Table get_table(1:string dbname, 2:string tbl_name)
>                        throws (1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Table> get_table_objects_by_name(1:string dbname, 2:list<string> tbl_names)
>            throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   list<string> get_table_names_by_filter(1:string dbname, 2:string filter, 3:i16 max_tables=-1)
>                        throws (1:MetaException o1, 2:InvalidOperationException o2, 3:UnknownDBException o3)
>   Partition add_partition(1:Partition new_part)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   i32 add_partitions(1:list<Partition> new_parts)
>                        throws(1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   Partition append_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name)
>                        throws (1:InvalidObjectException o1, 2:AlreadyExistsException o2, 3:MetaException o3)
>   bool drop_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   bool drop_partition_by_name(1:string db_name, 2:string tbl_name, 3:string part_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Partition get_partition(1:string db_name, 2:string tbl_name, 3:list<string> part_vals)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals,
>       4: string user_name, 5: list<string> group_names) throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   Partition get_partition_by_name(1:string db_name 2:string tbl_name, 3:string part_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<Partition> get_partitions_with_auth(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1,
>      4: string user_name, 5: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names(1:string db_name, 2:string tbl_name, 3:i16 max_parts=-1)
>                        throws(1:MetaException o2)
>   list<Partition> get_partitions_ps(1:string db_name 2:string tbl_name
>     3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_ps_with_auth(1:string db_name, 2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1,
>      5: string user_name, 6: list<string> group_names) throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_partition_names_ps(1:string db_name,
>     2:string tbl_name, 3:list<string> part_vals, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_filter(1:string db_name 2:string tbl_name
>     3:string filter, 4:i16 max_parts=-1)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Partition> get_partitions_by_names(1:string db_name 2:string tbl_name 3:list<string> names)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   bool drop_index_by_name(1:string db_name, 2:string tbl_name, 3:string index_name, 4:bool deleteData)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   Index get_index_by_name(1:string db_name 2:string tbl_name, 3:string index_name)
>                        throws(1:MetaException o1, 2:NoSuchObjectException o2)
>   list<Index> get_indexes(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:NoSuchObjectException o1, 2:MetaException o2)
>   list<string> get_index_names(1:string db_name, 2:string tbl_name, 3:i16 max_indexes=-1)
>                        throws(1:MetaException o2)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira