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);
}