wuwenchi commented on pull request #4162: URL: https://github.com/apache/iceberg/pull/4162#issuecomment-1046759080
> Thanks for posting this @wuwenchi!This seems rather important. Can you please get the Spark2 tests working? It seems related. > > It would also be very helpful if you could add a test to demonstrate that this fixes the issue (something that breaks before that doesn't break after). The test case already exists, called testRewriteAvoidRepeatCompress. However, there are some problems in the writing of this test case, which leads to the fact that the offset list of parquet is not updated, and the wrong file length is obtained. The wrong file length, coupled with the wrong judgment process, has obtained the expected correct result. So I directly modify this test case. -- 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]
