You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by xu...@apache.org on 2015/01/08 15:33:49 UTC
svn commit: r1650296 -
/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java
Author: xuefu
Date: Thu Jan 8 14:33:48 2015
New Revision: 1650296
URL: http://svn.apache.org/r1650296
Log:
HIVE-9301: Potential null dereference in MoveTask#createTargetPath (Ted via Xuefu)
Modified:
hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java
Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java?rev=1650296&r1=1650295&r2=1650296&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java Thu Jan 8 14:33:48 2015
@@ -145,7 +145,7 @@ public class MoveTask extends Task<MoveW
private Path createTargetPath(Path targetPath, FileSystem fs) throws IOException {
Path deletePath = null;
Path mkDirPath = targetPath.getParent();
- if (mkDirPath != null & !fs.exists(mkDirPath)) {
+ if (mkDirPath != null && !fs.exists(mkDirPath)) {
Path actualPath = mkDirPath;
// targetPath path is /x/y/z/1/2/3 here /x/y/z is present in the file system
// create the structure till /x/y/z/1/2 to work rename for multilevel directory