hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hive QA (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-16943) MoveTask should separate src FileSystem from dest FileSystem
Date Fri, 23 Jun 2017 05:02:01 GMT

    [ https://issues.apache.org/jira/browse/HIVE-16943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16060405#comment-16060405
] 

Hive QA commented on HIVE-16943:
--------------------------------



Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12874172/HIVE-16943.1.patch

{color:red}ERROR:{color} -1 due to no test(s) being added or modified.

{color:red}ERROR:{color} -1 due to 20 failed/errored test(s), 10730 tests executed
*Failed tests:*
{noformat}
TestCleaner2 - did not produce a TEST-*.xml file (likely timed out) (batchId=258)
TestConvertAstToSearchArg - did not produce a TEST-*.xml file (likely timed out) (batchId=258)
TestIOContextMap - did not produce a TEST-*.xml file (likely timed out) (batchId=258)
TestInitiator - did not produce a TEST-*.xml file (likely timed out) (batchId=258)
TestRecordIdentifier - did not produce a TEST-*.xml file (likely timed out) (batchId=258)
TestSearchArgumentImpl - did not produce a TEST-*.xml file (likely timed out) (batchId=258)
TestWorker - did not produce a TEST-*.xml file (likely timed out) (batchId=258)
TestWorker2 - did not produce a TEST-*.xml file (likely timed out) (batchId=258)
org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[tez_smb_main] (batchId=150)
org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[vector_if_expr] (batchId=146)
org.apache.hadoop.hive.cli.TestPerfCliDriver.testCliDriver[query14] (batchId=233)
org.apache.hadoop.hive.cli.TestPerfCliDriver.testCliDriver[query16] (batchId=233)
org.apache.hadoop.hive.cli.TestPerfCliDriver.testCliDriver[query94] (batchId=233)
org.apache.hadoop.hive.cli.TestSparkCliDriver.testCliDriver[union24] (batchId=125)
org.apache.hadoop.hive.ql.parse.TestReplicationScenariosAcrossInstances.testBootstrapFunctionReplication
(batchId=217)
org.apache.hadoop.hive.ql.parse.TestReplicationScenariosAcrossInstances.testCreateFunctionIncrementalReplication
(batchId=217)
org.apache.hadoop.hive.ql.parse.TestReplicationScenariosAcrossInstances.testCreateFunctionWithFunctionBinaryJarsOnHDFS
(batchId=217)
org.apache.hive.hcatalog.api.TestHCatClient.testPartitionRegistrationWithCustomSchema (batchId=178)
org.apache.hive.hcatalog.api.TestHCatClient.testPartitionSpecRegistrationWithCustomSchema
(batchId=178)
org.apache.hive.hcatalog.api.TestHCatClient.testTableSchemaPropagation (batchId=178)
{noformat}

Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/5738/testReport
Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/5738/console
Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-5738/

Messages:
{noformat}
Executing org.apache.hive.ptest.execution.TestCheckPhase
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
Tests exited with: TestsFailedException: 20 tests failed
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12874172 - PreCommit-HIVE-Build

> MoveTask should separate src FileSystem from dest FileSystem 
> -------------------------------------------------------------
>
>                 Key: HIVE-16943
>                 URL: https://issues.apache.org/jira/browse/HIVE-16943
>             Project: Hive
>          Issue Type: Bug
>          Components: Query Processor
>    Affects Versions: 3.0.0
>            Reporter: Fei Hui
>            Assignee: Fei Hui
>         Attachments: HIVE-16943.1.patch
>
>
> {code:title=MoveTask.java|borderStyle=solid} 
>  private void moveFileInDfs (Path sourcePath, Path targetPath, FileSystem fs)
>       throws HiveException, IOException {
>     // if source exists, rename. Otherwise, create a empty directory
>     if (fs.exists(sourcePath)) {
>       Path deletePath = null;
>       // If it multiple level of folder are there fs.rename is failing so first
>       // create the targetpath.getParent() if it not exist
>       if (HiveConf.getBoolVar(conf, HiveConf.ConfVars.HIVE_INSERT_INTO_MULTILEVEL_DIRS))
{
>         deletePath = createTargetPath(targetPath, fs);
>       }
>       Hive.clearDestForSubDirSrc(conf, targetPath, sourcePath, false);
>       if (!Hive.moveFile(conf, sourcePath, targetPath, true, false)) {
>         try {
>           if (deletePath != null) {
>             fs.delete(deletePath, true);
>           }
>         } catch (IOException e) {
>           LOG.info("Unable to delete the path created for facilitating rename"
>               + deletePath);
>         }
>         throw new HiveException("Unable to rename: " + sourcePath
>             + " to: " + targetPath);
>       }
>     } else if (!fs.mkdirs(targetPath)) {
>       throw new HiveException("Unable to make directory: " + targetPath);
>     }
>   }
> {code}
> Maybe sourcePath and targetPath come from defferent filesystem, we should separate them.
> I see that HIVE-11568 had done it in Hive.java



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message