kbendick commented on PR #4786: URL: https://github.com/apache/iceberg/pull/4786#issuecomment-1132133637
> I think I found the issue (which can easily be debugged locally by adding `--no-parallel --debug` to get `./gradlew -DsparkVersions= -DhiveVersions= -DflinkVersions=1.13 :iceberg-flink:iceberg-flink-1.13:check :iceberg-flink:iceberg-flink-runtime-1.13:check -Pquick=true -x javadoc --no-parallel --debug`). It looks like something got messed up during conflict resolution and `BaseDeltaTaskWriter#asStructLike(..)` was accidentally returning `keyWrapper.wrap(data)` and not `wrapper.wrap(data)`. This then resulted in `TestChangeLogTable#testChangeLogOnDataKey()` to run indefinitely in a retry-loop because there was an underlying exception. For the future I think it would be good to have a timeout on tests as issues like these are quite difficult to debug on CI without modifying how the test is being executed (e.g. by adding `--debug` to the gradle task and such) It's possible there's somethign else we need to cherry-pick given the the underlying exception for that one test case. I'll run locally and we'll see. -- 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]
