richardkerr commented on PR #1048: URL: https://github.com/apache/parquet-mr/pull/1048#issuecomment-1500348451
> My PR adds the following unit tests OR does not need testing for this extremely good reason: On this, I was struggling to put together a test that was able to replicate the issue without extensive changes to the writers - if anyone has insight on a good way to do that please make the change. Otherwise I feel the change is trivial enough that a unit test is not necessarily worth the effort and FWIW I have verified it using externally generated files. -- 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]
