felipepessoto commented on code in PR #13418: URL: https://github.com/apache/gluten/pull/13418#discussion_r4362890407
########## .github/workflows/util/delta-spark-ut/flaky-tests.txt: ########## @@ -20,3 +20,10 @@ # whack-a-mole), quarantine it by ERROR SIGNATURE in flaky-error-patterns.txt # instead. +# Timing-sensitive: the test gives an AvailableNow-triggered CDC stream 10s to +# terminate (query.awaitTermination(10000L)). With 4 forked JVMs near the runner's +# memory limit, the batch can still be in Gluten's Parquet metadata validation +# (ParquetMetadataUtils listing files via RawLocalFileSystem, which forks `ls` per +# file for permissions), so it times out. Failed once in 25 runs, on the +# Velox-bump PR #13418 (run 39902045267), which changed no Java code. Review Comment: @zhouyuan I have some other improvements here https://github.com/apache/gluten/pull/13129. The PR is old so I think it got lost in the list of active PRs and reviewers missed it -- 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]
