zeroshade commented on issue #451: URL: https://github.com/apache/arrow-go/issues/451#issuecomment-3120017148
It's been a consistent thorn in my side that I've waffled on whether or not I should remove that `io.EOF` return or just improve the other readers. At a minimum, this shouldn't fail this way so it's definitely a bug. Looking through the code and documentation, I think it would probably be best to fix the record reader in `pqarrow`. I think it's set up to return the `io.EOF` to make the implementation of the `Read` method (for `arrio.Reader`) easier. I can try to find some time to fix this, but I'm going to be away for the end of next week so I probably won't find time for a couple weeks. Would you be willing to contribute a fix? I'll happily review and get it in -- 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. To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
