alamb commented on PR #100: URL: https://github.com/apache/parquet-testing/pull/100#issuecomment-4351940914
Sorry I didn't post this earlier. My biggest concern with this PR is that it adds many bytes of example data. Since this repo is checked out many times by many CI jobs (e.g. arrow, arrow-rs, etc) I think it is important to keep the size down Furthermore, I am not sure that this covers the major parameter (I need to review the files carefully first) I will file a ticket that records what I think is needed, and link this PR -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
