hive icon indicating copy to clipboard operation
hive copied to clipboard

HIVE-28350: Drop remote database should succeed deleting data under d…

Open saihemanth-cloudera opened this issue 1 year ago • 1 comments

…b location

What changes were proposed in this pull request?

Setting remote database location and managed location

Why are the changes needed?

Otherwise drop remote database is not successful

Does this PR introduce any user-facing change?

No

Is the change a dependency upgrade?

No

How was this patch tested?

Local Cluster

saihemanth-cloudera avatar Jun 25 '24 20:06 saihemanth-cloudera

I'm not sure how the fix can solve the problem, might the mdb.getLocationUri() or mdb.getManagedLocationUri() is null for remote database?

dengzhhu653 avatar Jul 19 '24 01:07 dengzhhu653

I'm not sure how the fix can solve the problem, might the mdb.getLocationUri() or mdb.getManagedLocationUri() is null for remote database?

Yeah, because the location URI is null, we would hit null pointer exception here: https://github.com/apache/hive/blob/master/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java#L1803-L1804

saihemanth-cloudera avatar Jul 19 '24 21:07 saihemanth-cloudera