orc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From majetideepak <...@git.apache.org>
Subject [GitHub] orc pull request #277: ORC-372: Enable valgrind for C++ travis-ci tests
Date Tue, 05 Jun 2018 00:48:02 GMT
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


---

Mime
View raw message