jecsand838 commented on PR #8220: URL: https://github.com/apache/arrow-rs/pull/8220#issuecomment-3225293278
> THank you @jecsand838 -- the code looks good to me. I am a little worried about lack of test coverage -- is there any chance you can add coverage for skipping more of the types? > > I think "end to end" type skipping tests would be the best. Maybe something like > > 1. Write a file with all supported avro types > 2. Read each (single) column back (skipping all the others) > 3. Verify the output column is the same as was written. That's a good callout! I can definitely do that. -- 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
