[ 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