HIVE-25500: Switch back to alter_partition(s) in HMS client for Hive 2.3.x

[cxzl25]

What changes were proposed in this pull request?

Use alter_partition, alter_partitions when EnvironmentContext is null.

Why are the changes needed?

HIVE-12730 uses alter_partition_with_environment_context and alter_partitions_with_environment_context instead of alter_partition and alter_partitions when the EnvironmentContext is null.
This causes the client version to be greater than 2.1 to connect to server<2.1, and this error will occur.

Caused by: org.apache.thrift.TApplicationException: Invalid method name: 'alter_partitions_with_environment_context'
	at org.apache.thrift.TServiceClient.receiveBase(TServiceClient.java:79)
	at org.apache.hadoop.hive.metastore.api.ThriftHiveMetastore$Client.recv_alter_partitions_with_environment_context(ThriftHiveMetastore.java:2843)
	at org.apache.hadoop.hive.metastore.api.ThriftHiveMetastore$Client.alter_partitions_with_environment_context(ThriftHiveMetastore.java:2827)
	at org.apache.hadoop.hive.metastore.HiveMetaStoreClient.alter_partitions(HiveMetaStoreClient.java:1524)

Does this PR introduce any user-facing change?

No.

How was this patch tested?

exist UT

[cxzl25]

@sunchao Help review when you have time. Thanks.

[sunchao]

Thanks @cxzl25 ! this is a good catch and it LGTM. Could you open another one for branch-3.1 as well? Ideally we should also fix this in master but seems the code path has changed a lot there.

[cxzl25]

Thanks @cxzl25 ! this is a good catch and it LGTM. Could you open another one for branch-3.1 as well? Ideally we should also fix this in master but seems the code path has changed a lot there.

Submitted a PR (#2623) on branch3.1 .
master branch: HIVE-19820 uses alter_partitions_req instead of alter_partitions .

[sunchao]

Merged to branch-2.3 Thanks @cxzl25 !