minor debugging
Project: http://git-wip-us.apache.org/repos/asf/orc/repo Commit: http://git-wip-us.apache.org/repos/asf/orc/commit/2246b1a0 Tree: http://git-wip-us.apache.org/repos/asf/orc/tree/2246b1a0 Diff: http://git-wip-us.apache.org/repos/asf/orc/diff/2246b1a0 Branch: refs/heads/orc-72 Commit: 2246b1a01a26e01e72b31047f85a81c0cf64f08f Parents: 839d29d Author: Owen O'Malley <omal...@apache.org> Authored: Fri Oct 14 13:06:36 2016 -0700 Committer: Owen O'Malley <omal...@apache.org> Committed: Fri Oct 14 13:07:23 2016 -0700 ---------------------------------------------------------------------- .../java/org/apache/orc/bench/avro/AvroSchemaUtils.java | 12 +++++++----- .../src/java/org/apache/orc/bench/avro/AvroWriter.java | 2 ++ 2 files changed, 9 insertions(+), 5 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/orc/blob/2246b1a0/java/bench/src/java/org/apache/orc/bench/avro/AvroSchemaUtils.java ---------------------------------------------------------------------- diff --git a/java/bench/src/java/org/apache/orc/bench/avro/AvroSchemaUtils.java b/java/bench/src/java/org/apache/orc/bench/avro/AvroSchemaUtils.java index 5df7b70..893a4fd 100644 --- a/java/bench/src/java/org/apache/orc/bench/avro/AvroSchemaUtils.java +++ b/java/bench/src/java/org/apache/orc/bench/avro/AvroSchemaUtils.java @@ -110,7 +110,7 @@ public class AvroSchemaUtils { throw new UnsupportedOperationException(typeInfo + " is not supported."); } - return wrapInUnionWithNull(schema); + return schema; } private static Schema createAvroUnion(TypeDescription typeInfo) { @@ -128,7 +128,7 @@ public class AvroSchemaUtils { } } - return Schema.createUnion(childSchemas); + return wrapInUnionWithNull(Schema.createUnion(childSchemas)); } private static Schema createAvroRecord(TypeDescription typeInfo) { @@ -140,7 +140,8 @@ public class AvroSchemaUtils { for (int i = 0; i < fieldNames.size(); ++i) { TypeDescription childTypeInfo = fieldTypes.get(i); Schema.Field field = new Schema.Field(fieldNames.get(i), - createAvroSchema(childTypeInfo), childTypeInfo.toString(), + wrapInUnionWithNull(createAvroSchema(childTypeInfo)), + childTypeInfo.toString(), (Object) null); childFields.add(field); } @@ -158,14 +159,15 @@ public class AvroSchemaUtils { + typeInfo); } - Schema valueSchema = createAvroSchema(typeInfo.getChildren().get(1)); + Schema valueSchema = wrapInUnionWithNull(createAvroSchema + (typeInfo.getChildren().get(1))); return Schema.createMap(valueSchema); } private static Schema createAvroArray(TypeDescription typeInfo) { Schema child = createAvroSchema(typeInfo.getChildren().get(0)); - return Schema.createArray(child); + return Schema.createArray(wrapInUnionWithNull(child)); } private static Schema wrapInUnionWithNull(Schema schema) { http://git-wip-us.apache.org/repos/asf/orc/blob/2246b1a0/java/bench/src/java/org/apache/orc/bench/avro/AvroWriter.java ---------------------------------------------------------------------- diff --git a/java/bench/src/java/org/apache/orc/bench/avro/AvroWriter.java b/java/bench/src/java/org/apache/orc/bench/avro/AvroWriter.java index 2735a71..8cc9d06 100644 --- a/java/bench/src/java/org/apache/orc/bench/avro/AvroWriter.java +++ b/java/bench/src/java/org/apache/orc/bench/avro/AvroWriter.java @@ -308,6 +308,8 @@ public class AvroWriter { String compression) throws IOException { List<TypeDescription> childTypes = schema.getChildren(); Schema avroSchema = AvroSchemaUtils.createAvroSchema(schema); + System.out.println("Hive schema " + schema); + System.out.println("Avro schema " + avroSchema); List<Schema.Field> avroFields = avroSchema.getFields(); converters = new AvroConverter[childTypes.size()]; for(int c=0; c < converters.length; ++c) {