HIVE-26012: HMS APIs to be enhanced for metadata replication

[DanielZhu58]

What changes were proposed in this pull request?

Change the CreateDatabaseRequest struct, to add a boolean value skipFSWrites. This boolean value decides whether to create the directory of this database in file system or not. Same as CreateTableRequest struct and AddPartitionsRequest struct. Meanwhile, enhance the HMS API to accept additional boolean skipFSWrites parameters to skip the creation on a need-to basis.

Why are the changes needed?

The following DDL statements in hive result in creation of a directories on the DFS filesystem. "create database", "create table", "alter table 'table_name' add partition" This is done to ensure that the user (be it be the service user or end user) has the privileges to access the underlying filesystem location. But it also sets up the subsequent queries from not having the burden of creating the directories “on demand”. For example, having the execution for “create table” statement also create the database root directories would be a little out of place. Making this change can let the users to skip the creation on a need-to basis.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Unit tests, Thrift build, Maven build

[dengzhhu653]

also should we test renaming a table that skips creating the directory?

[saihemanth-cloudera]

@DanielZhu58 - Can you post the replies for @dengzhhu653's questions? Thanks.

[sonarqubecloud[bot]]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
19 Code Smells

No Coverage information
No Duplication information

[github-actions[bot]]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Feel free to reach out on the [email protected] list if the patch is in need of reviews.