Hi Wechar, 

The proposed signature covers a wider use case, making the existing one no 
longer needed

new:
 List<Partition> dropPartitions(String dbName, String tblName,
    RequestPartsSpec partsSpec, 
    PartitionDropOptions options)

existing:
List<Partition> dropPartitions(String dbName, String tblName,
    List<Pair<Integer, byte[]>> partExprs,
    PartitionDropOptions options)

however, can we wrap everything into the `DropPartitionsRequest` instead of 
introducing 2 API methods: one for default catalog and one with catalog param? 

Regards,
Denys

Reply via email to