svn commit: r1427208 - in /incubator/hcatalog/trunk: CHANGES.txt core/src/main/java/org/apache/hcatalog/mapreduce/FileOutputCommitterContainer.java

[ga...@apache.org]

Author: gates
Date: Mon Dec 31 17:56:59 2012
New Revision: 1427208

URL: http://svn.apache.org/viewvc?rev=1427208&view=rev
Log:
HCATALOG-584 Changes in HCAT-538 break Pig stores into non-partitioned tables

Modified:
    incubator/hcatalog/trunk/CHANGES.txt
    incubator/hcatalog/trunk/core/src/main/java/org/apache/hcatalog/mapreduce/FileOutputCommitterContainer.java

Modified: incubator/hcatalog/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/incubator/hcatalog/trunk/CHANGES.txt?rev=1427208&r1=1427207&r2=1427208&view=diff
==============================================================================
--- incubator/hcatalog/trunk/CHANGES.txt (original)
+++ incubator/hcatalog/trunk/CHANGES.txt Mon Dec 31 17:56:59 2012
@@ -168,6 +168,8 @@ Release 0.5.0 - Unreleased
   OPTIMIZATIONS
 
   BUG FIXES
+  HCAT-584 Changes in HCAT-538 break Pig stores into non-partitioned tables (gates)
+
   HCAT-576 HCATALOG-527 Broke backward compatibility (gates)
 
   HCAT-567 HCatClient must allow retrieval of multiple partitions using a partial partition spec.(mithun via avandana)

Modified: incubator/hcatalog/trunk/core/src/main/java/org/apache/hcatalog/mapreduce/FileOutputCommitterContainer.java
URL: http://svn.apache.org/viewvc/incubator/hcatalog/trunk/core/src/main/java/org/apache/hcatalog/mapreduce/FileOutputCommitterContainer.java?rev=1427208&r1=1427207&r2=1427208&view=diff
==============================================================================
--- incubator/hcatalog/trunk/core/src/main/java/org/apache/hcatalog/mapreduce/FileOutputCommitterContainer.java (original)
+++ incubator/hcatalog/trunk/core/src/main/java/org/apache/hcatalog/mapreduce/FileOutputCommitterContainer.java Mon Dec 31 17:56:59 2012
@@ -420,6 +420,9 @@ class FileOutputCommitterContainer exten
                 if(LOG.isDebugEnabled()) {
                     LOG.debug("Moving file: [ " + file + "] to [" + finalOutputPath + "]");
                 }
+                // Make sure the parent directory exists.  It is not an error
+                // to recreate an existing directory
+                fs.mkdirs(finalOutputPath.getParent());
                 if (!fs.rename(file, finalOutputPath)) {
                     if (!fs.delete(finalOutputPath, true)) {
                         throw new HCatException(ErrorType.ERROR_MOVE_FAILED, "Failed to delete existing path " + finalOutputPath);