Github user majetideepak commented on a diff in the pull request:
https://github.com/apache/orc/pull/277#discussion_r192919379
--- Diff: c++/src/TypeImpl.cc ---
@@ -258,31 +258,34 @@ namespace orc {
case STRUCT: {
StructVectorBatch *result = new StructVectorBatch(capacity,
memoryPool);
+ std::unique_ptr<ColumnVectorBatch> return_value =
std::unique_ptr<ColumnVectorBatch>(result);
--- End diff --
we need `result` of type `StructVectorBatch` to access fields on line 263---
