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


---

Reply via email to