[
https://issues.apache.org/jira/browse/HIVE-25935?focusedWorklogId=735885&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-735885
]
ASF GitHub Bot logged work on HIVE-25935:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 03/Mar/22 11:45
Start Date: 03/Mar/22 11:45
Worklog Time Spent: 10m
Work Description: dengzhhu653 commented on a change in pull request #3072:
URL: https://github.com/apache/hive/pull/3072#discussion_r818573704
##########
File path:
standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java
##########
@@ -1545,83 +1545,6 @@ GetPartitionsPsWithAuthResponse
listPartitionsWithAuthInfoRequest(GetPartitionsP
PartitionsResponse getPartitionsRequest(PartitionsRequest req)
throws NoSuchObjectException, MetaException, TException;
- /**
- * Get partitions by a list of partition names.
- * @param db_name database name
- * @param tbl_name table name
- * @param part_names list of partition names
- * @param getColStats if true include statistics in the Partition object
- * @param engine engine sending the request
- * @return list of Partition objects
- * @throws NoSuchObjectException No such partitionscatName
- * @throws MetaException error accessing the RDBMS.
- * @throws TException thrift transport error
- */
- List<Partition> getPartitionsByNames(String db_name, String tbl_name,
List<String> part_names,
- boolean getColStats, String engine) throws NoSuchObjectException,
MetaException, TException;
-
- /**
- * Get partitions by a list of partition names.
- * @param catName catalog name
- * @param db_name database name
- * @param tbl_name table name
- * @param part_names list of partition names
- * @return list of Partition objects
- * @throws NoSuchObjectException No such partitions
- * @throws MetaException error accessing the RDBMS.
- * @throws TException thrift transport error
- */
- List<Partition> getPartitionsByNames(String catName, String db_name, String
tbl_name,
Review comment:
Seems this method is released from version 3.0.0
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 735885)
Time Spent: 40m (was: 0.5h)
> Cleanup IMetaStoreClient#getPartitionsByNames APIs
> --------------------------------------------------
>
> Key: HIVE-25935
> URL: https://issues.apache.org/jira/browse/HIVE-25935
> Project: Hive
> Issue Type: Task
> Components: Metastore
> Reporter: Stamatis Zampetakis
> Assignee: Peter Vary
> Priority: Major
> Labels: pull-request-available
> Fix For: 4.0.0-alpha-1
>
> Time Spent: 40m
> Remaining Estimate: 0h
>
> Currently the
> [IMetastoreClient|https://github.com/apache/hive/blob/4b7a948e45fd88372fef573be321cda40d189cc7/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java]
> interface has 8 variants of the {{getPartitionsByNames}} method. Going
> quickly over the concrete implementation it appears that not all of them are
> useful/necessary so a bit of cleanup is needed.
> Below a few potential problems I observed:
> * Some of the APIs are not used anywhere in the project (neither by
> production nor by test code).
> * Some of the APIs are deprecated in some concrete implementations but not
> globally at the interface level without an explanation why.
> * Some of the implementations simply throw without doing anything.
> * Many of the APIs are partially tested or not tested at all.
> HIVE-24743, HIVE-25281 are related since they introduce/deprecate some of the
> aforementioned APIs.
> It would be good to review the aforementioned APIs and decide what needs to
> stay and what needs to go as well as complete necessary when relevant.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
