-
Notifications
You must be signed in to change notification settings - Fork 2.5k
[HUDI-5630] Fixing flaky parquet projection tests #7768
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
...ource/hudi-spark/src/test/scala/org/apache/hudi/functional/TestParquetColumnProjection.scala
Outdated
Show resolved
Hide resolved
...ource/hudi-spark/src/test/scala/org/apache/hudi/functional/TestParquetColumnProjection.scala
Outdated
Show resolved
Hide resolved
xushiyan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
enable other skipped cases? those set to -1
|
Failed due to a flaky deltastreamer test. going ahead to unblock other blocker patches |
Fixing flaky parquet projection tests. Added 10% margin for expected bytes from col projection.
Fixing flaky parquet projection tests. Added 10% margin for expected bytes from col projection.
| package org.apache.hudi.functional | ||
|
|
||
| import org.apache.avro.Schema | ||
| import org.apache.calcite.runtime.SqlFunctions.abs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't Math.abs be sufficient?
Fixing flaky parquet projection tests. Added 10% margin for expected bytes from col projection.
Change Logs
Fixing flaky parquet projection tests. Added 10% margin for expected bytes from col projection.
Impact
Stable CI
Risk level (write none, low medium or high below)
none
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change
ticket number here and follow the instruction to make
changes to the website.
Contributor's checklist