You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@arrow.apache.org by "Wes McKinney (JIRA)" <ji...@apache.org> on 2018/11/24 17:11:00 UTC

[jira] [Created] (ARROW-3870) [C++] Add Peek to InputStream API

Wes McKinney created ARROW-3870:
-----------------------------------

             Summary: [C++] Add Peek to InputStream API
                 Key: ARROW-3870
                 URL: https://issues.apache.org/jira/browse/ARROW-3870
             Project: Apache Arrow
          Issue Type: Improvement
          Components: C++
            Reporter: Wes McKinney
             Fix For: 0.12.0


Some InputStream sources support lookahead without advancing the stream, like BufferReader and BufferedInputStream. This is used in Parquet's internal IO APIs, so it would be useful to be able to remove these after a deprecation cycle

https://github.com/apache/arrow/blob/master/cpp/src/parquet/util/memory.h#L375

If an InputStream does not support Peek, the returned {{string_view}} would have zero size. 

Another option is to have {{Peek}} return {{Status}}, so that NotImplemented can be returned. I would prefer to return 0-length if the operation is not supported



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)