bkietz commented on pull request #8052: URL: https://github.com/apache/arrow/pull/8052#issuecomment-696303007
Rewinding doesn't strike me as something which needs to be part of the C stream protocol. APIs can still provide rewind and other semantics while using a simple-as-possible stream as a building block. In the case of a SQL view which can be viewed multiple times for example, let the SQL view need not be a stream. It could instead be a function returning streams (each beginning at the start of the view): ```diff -ArrowArrayStream* MakeSqlView(...); +SqlView* MakeSqlView(...); +ArrowArrayStream* GetStream(SqlView*); ``` Rewind, scrolling, offsets, reverse iteration, etc can all be accommodated in this fashion so IMHO they don't belong in the protocol. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
