You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hcatalog-commits@incubator.apache.org by tr...@apache.org on 2012/07/27 00:10:41 UTC

svn commit: r1366243 - in /incubator/hcatalog/branches/branch-0.4: CHANGES.txt src/java/org/apache/hcatalog/pig/HCatLoader.java src/java/org/apache/hcatalog/pig/HCatStorer.java

Author: travis
Date: Fri Jul 27 00:10:41 2012
New Revision: 1366243

URL: http://svn.apache.org/viewvc?rev=1366243&view=rev
Log:
HCATALOG-449 : HCatLoader is mistakenly identifying Configuration parameters to store

Modified:
    incubator/hcatalog/branches/branch-0.4/CHANGES.txt
    incubator/hcatalog/branches/branch-0.4/src/java/org/apache/hcatalog/pig/HCatLoader.java
    incubator/hcatalog/branches/branch-0.4/src/java/org/apache/hcatalog/pig/HCatStorer.java

Modified: incubator/hcatalog/branches/branch-0.4/CHANGES.txt
URL: http://svn.apache.org/viewvc/incubator/hcatalog/branches/branch-0.4/CHANGES.txt?rev=1366243&r1=1366242&r2=1366243&view=diff
==============================================================================
--- incubator/hcatalog/branches/branch-0.4/CHANGES.txt (original)
+++ incubator/hcatalog/branches/branch-0.4/CHANGES.txt Fri Jul 27 00:10:41 2012
@@ -51,6 +51,8 @@ Trunk (unreleased changes)
   OPTIMIZATIONS
 
   BUG FIXES
+  HCAT-449 HCatLoader is mistakenly identifying Configuration parameters to store (cdrome via traviscrawford)
+
   HCAT-452 HCat_Drop_Table_3 does not get initialized properly (cdrome via toffer)  
 
   HCAT-448 HCatStorer performance is 4x slower in HCat 0.4 than HCat 0.2 (rajesh.balamohan via khorgath)

Modified: incubator/hcatalog/branches/branch-0.4/src/java/org/apache/hcatalog/pig/HCatLoader.java
URL: http://svn.apache.org/viewvc/incubator/hcatalog/branches/branch-0.4/src/java/org/apache/hcatalog/pig/HCatLoader.java?rev=1366243&r1=1366242&r2=1366243&view=diff
==============================================================================
--- incubator/hcatalog/branches/branch-0.4/src/java/org/apache/hcatalog/pig/HCatLoader.java (original)
+++ incubator/hcatalog/branches/branch-0.4/src/java/org/apache/hcatalog/pig/HCatLoader.java Fri Jul 27 00:10:41 2012
@@ -119,7 +119,7 @@ public class HCatLoader extends HCatBase
             // udf context, so the the HCatInputFormat.setInput method need not
             //be called many times.
             for (Entry<String,String> keyValue : job.getConfiguration()) {
-                String oldValue = clone.getConfiguration().get(keyValue.getKey());
+                String oldValue = clone.getConfiguration().getRaw(keyValue.getKey());
                 if ((oldValue == null) || (keyValue.getValue().equals(oldValue) == false)) {
                     udfProps.put(keyValue.getKey(), keyValue.getValue());
                 }

Modified: incubator/hcatalog/branches/branch-0.4/src/java/org/apache/hcatalog/pig/HCatStorer.java
URL: http://svn.apache.org/viewvc/incubator/hcatalog/branches/branch-0.4/src/java/org/apache/hcatalog/pig/HCatStorer.java?rev=1366243&r1=1366242&r2=1366243&view=diff
==============================================================================
--- incubator/hcatalog/branches/branch-0.4/src/java/org/apache/hcatalog/pig/HCatStorer.java (original)
+++ incubator/hcatalog/branches/branch-0.4/src/java/org/apache/hcatalog/pig/HCatStorer.java Fri Jul 27 00:10:41 2012
@@ -136,7 +136,7 @@ public class HCatStorer extends HCatBase
       // udf context, so the the HCatOutputFormat.setOutput and setSchema
       // methods need not be called many times.
       for ( Entry<String,String> keyValue : job.getConfiguration()) {
-        String oldValue = clone.getConfiguration().get(keyValue.getKey());
+        String oldValue = clone.getConfiguration().getRaw(keyValue.getKey());
         if ((oldValue == null) || (keyValue.getValue().equals(oldValue) == false)) {
           udfProps.put(keyValue.getKey(), keyValue.getValue());
         }