bkietz edited a comment on pull request #8256: URL: https://github.com/apache/arrow/pull/8256#issuecomment-724995201
[Another sort of failure you might see:](https://github.com/apache/arrow/pull/8256/checks?check_run_id=1382178751#step:8:3439) ``` ── 5. Failure: RecordBatchStreamReader / Writer (@test-record-batch-reader.R#31) `writer` inherits from `RecordBatchWriter/ArrowObject/R6` not `RecordBatchStreamWriter`. ``` In R there are separate R6 classes `/RecordBatch(|Stream|File)Writer/` but in c++ there's only `RecordBatchWriter`. It does not provide any property which would tell you whether it's writing to a stream or a file so there's no way we can tell what the right R6 class would be. In this instance, it's a clue that we should probably delete the extra R6 classes (leaving only RecordBatchWriter) since they're not doing anything except holding a `$create` ---------------------------------------------------------------- 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: [email protected]
