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 to...@apache.org on 2012/05/01 20:40:23 UTC

svn commit: r1332807 - in /incubator/hcatalog/trunk/src/java/org/apache/hcatalog/data: HCatRecordSerDe.java JsonSerDe.java

Author: toffer
Date: Tue May  1 20:40:22 2012
New Revision: 1332807

URL: http://svn.apache.org/viewvc?rev=1332807&view=rev
Log:
HCATALOG-68 correct addendum to patch missed some changes in comments (avandana via toffer)

Modified:
    incubator/hcatalog/trunk/src/java/org/apache/hcatalog/data/HCatRecordSerDe.java
    incubator/hcatalog/trunk/src/java/org/apache/hcatalog/data/JsonSerDe.java

Modified: incubator/hcatalog/trunk/src/java/org/apache/hcatalog/data/HCatRecordSerDe.java
URL: http://svn.apache.org/viewvc/incubator/hcatalog/trunk/src/java/org/apache/hcatalog/data/HCatRecordSerDe.java?rev=1332807&r1=1332806&r2=1332807&view=diff
==============================================================================
--- incubator/hcatalog/trunk/src/java/org/apache/hcatalog/data/HCatRecordSerDe.java (original)
+++ incubator/hcatalog/trunk/src/java/org/apache/hcatalog/data/HCatRecordSerDe.java Tue May  1 20:40:22 2012
@@ -87,8 +87,8 @@ public class HCatRecordSerDe implements 
     }
 
 
-    LOG.debug("columns: {}",columnNameProperty,columnNames);
-    LOG.debug("types: {}", columnTypeProperty, columnTypes);
+    LOG.debug("columns: {} {}",columnNameProperty,columnNames);
+    LOG.debug("types: {} {}", columnTypeProperty, columnTypes);
     assert (columnNames.size() == columnTypes.size());
 
     rowTypeInfo = (StructTypeInfo) TypeInfoFactory.getStructTypeInfo(columnNames, columnTypes);

Modified: incubator/hcatalog/trunk/src/java/org/apache/hcatalog/data/JsonSerDe.java
URL: http://svn.apache.org/viewvc/incubator/hcatalog/trunk/src/java/org/apache/hcatalog/data/JsonSerDe.java?rev=1332807&r1=1332806&r2=1332807&view=diff
==============================================================================
--- incubator/hcatalog/trunk/src/java/org/apache/hcatalog/data/JsonSerDe.java (original)
+++ incubator/hcatalog/trunk/src/java/org/apache/hcatalog/data/JsonSerDe.java Tue May  1 20:40:22 2012
@@ -150,10 +150,12 @@ public class JsonSerDe implements SerDe 
         populateRecord(r,token,p,schema);
       }
     } catch (JsonParseException e) {
-        LOG.warn("Error [{}] parsing json location [{}].", e.getMessage(), e.getLocation());
+      LOG.warn("Error [{}] parsing json text [{}].", e, t);
+      LOG.debug(null,e);
       throw new SerDeException(e);
     } catch (IOException e) {
-        LOG.warn("Error [{}] parsing json text [{}].", e.getMessage(),t);
+      LOG.warn("Error [{}] parsing json text [{}].", e, t);
+      LOG.debug(null,e);
       throw new SerDeException(e);
     }