You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@orc.apache.org by ga...@apache.org on 2021/09/10 09:47:23 UTC

[orc] branch main updated: ORC-990: [C++] Fix undefined behavior of RowReaderImpl::seekToRowGroup

This is an automated email from the ASF dual-hosted git repository.

gangwu pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/orc.git


The following commit(s) were added to refs/heads/main by this push:
     new ebf33dc  ORC-990: [C++] Fix undefined behavior of RowReaderImpl::seekToRowGroup
ebf33dc is described below

commit ebf33dc229518e28b9d864cb83c63a7cb8d8ed61
Author: Renat Valiullin <ri...@gmail.com>
AuthorDate: Fri Sep 10 02:47:19 2021 -0700

    ORC-990: [C++] Fix undefined behavior of RowReaderImpl::seekToRowGroup
    
    This fixes #903
---
 c++/src/Reader.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/c++/src/Reader.cc b/c++/src/Reader.cc
index 0811dbe..c44439a 100644
--- a/c++/src/Reader.cc
+++ b/c++/src/Reader.cc
@@ -385,7 +385,7 @@ namespace orc {
 
   void RowReaderImpl::seekToRowGroup(uint32_t rowGroupEntryId) {
     // store positions for selected columns
-    std::vector<std::list<uint64_t>> positions;
+    std::list<std::list<uint64_t>> positions;
     // store position providers for selected colimns
     std::unordered_map<uint64_t, PositionProvider> positionProviders;