[core] Refactor MetastoreClient methods to simplify catalog

[JingsongLi]

Purpose

New MetastoreClient API:

/**
 * A metastore client related to a table. All methods of this interface operate on the same specific
 * table.
 */
public interface MetastoreClient extends AutoCloseable {

    void addPartition(LinkedHashMap<String, String> partition) throws Exception;

    void addPartitions(List<LinkedHashMap<String, String>> partitions) throws Exception;

    void dropPartition(LinkedHashMap<String, String> partition) throws Exception;

    void dropPartitions(List<LinkedHashMap<String, String>> partitions) throws Exception;

    void markPartitionDone(LinkedHashMap<String, String> partition) throws Exception;

    default void alterPartition(
            LinkedHashMap<String, String> partition, PartitionStats partitionStats)
            throws Exception {
        throw new UnsupportedOperationException();
    }

    /** Factory to create {@link MetastoreClient}. */
    interface Factory extends Serializable {

        MetastoreClient create();
    }
}

Tests

API and Format

Documentation

[wwj6591812]

addPartitions？not dropPartitions?

[wwj6591812]

modifyTimeInMillis

[JingsongLi]

See System.currentTimeMillis

[wwj6591812]

Map<String, String> statistic = new HashMap<>(5);

[JingsongLi]

For these small collection, we don't need to specific the init size.

[leaves12138]

+1