[ 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