[ 
https://issues.apache.org/jira/browse/HIVE-25331?focusedWorklogId=654367&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-654367
 ]

ASF GitHub Bot logged work on HIVE-25331:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 23/Sep/21 05:53
            Start Date: 23/Sep/21 05:53
    Worklog Time Spent: 10m 
      Work Description: pkumarsinha commented on a change in pull request #2478:
URL: https://github.com/apache/hive/pull/2478#discussion_r714488480



##########
File path: 
itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestReplicationScenariosExclusiveReplica.java
##########
@@ -710,7 +710,6 @@ private void verifyCustomDBLocations(String srcDb, 
List<String> listOfTables, St
               replicatedDbName.toLowerCase()  + ".db").toUri().getPath());
     } else {
       Assert.assertNotEquals(managedCustLocOnSrc,  null);
-      Assert.assertEquals(replDatabase.getManagedLocationUri(),  null);

Review comment:
       @ujc714  Rather than removing, I think we should assert that 
replDatabase.getManagedLocationUri() is in default warehouse location, i.e it 
should be equal to new Path(replica.warehouseRoot, 
replicatedDbName.toLowerCase()  + ".db")




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 654367)
    Time Spent: 2h  (was: 1h 50m)

> Create database query doesn't create MANAGEDLOCATION directory
> --------------------------------------------------------------
>
>                 Key: HIVE-25331
>                 URL: https://issues.apache.org/jira/browse/HIVE-25331
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Robbie Zhang
>            Assignee: Robbie Zhang
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 2h
>  Remaining Estimate: 0h
>
> If we don't assign MANAGEDLOCATION in a "create database" query, the 
> MANAGEDLOCATION will be NULL so HMS doesn't create the directory. In this 
> case, a CTAS query immediately after the CREATE DATABASE query might fail in 
> MOVE task due to "destination's parent does not exist". I can use the 
> following script to reproduce this issue:
> {code:java}
> set hive.support.concurrency=true;
> set hive.txn.manager=org.apache.hadoop.hive.ql.lockmgr.DbTxnManager;
> create database testdb location '/tmp/testdb.db';
> create table testdb.test as select 1;
> {code}
> If the staging directory is under the MANAGEDLOCATION directory, the CTAS 
> query is fine as the MANAGEDLOCATION directory is created while creating the 
> staging directory. Since we set LOCATION to a default directory when LOCATION 
> is not assigned in the CREATE DATABASE query, I believe it's worth to set 
> MANAGEDLOCATION to a default directory, too.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to