-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Add iceberg load test #31392
Add iceberg load test #31392
Conversation
Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #31392 +/- ##
=============================================
- Coverage 71.40% 68.55% -2.86%
- Complexity 1474 14921 +13447
=============================================
Files 900 2636 +1736
Lines 114166 222073 +107907
Branches 1076 11825 +10749
=============================================
+ Hits 81519 152234 +70715
- Misses 30619 63644 +33025
- Partials 2028 6195 +4167 ☔ View full report in Codecov by Sentry. |
Performance test ran successfully with the following jobs: Write (2024-05-28_14_38_09-6308924367920992234):
Read (2024-05-28_14_56_50-10726183500998931365):
|
R: @kennknowles |
Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control |
sdks/java/io/iceberg/src/test/java/org/apache/beam/sdk/io/iceberg/IcebergIOLT.java
Show resolved
Hide resolved
sdks/java/io/iceberg/src/test/java/org/apache/beam/sdk/io/iceberg/IcebergIOLT.java
Outdated
Show resolved
Hide resolved
sdks/java/io/iceberg/src/test/java/org/apache/beam/sdk/io/iceberg/IcebergIOLT.java
Show resolved
Hide resolved
sdks/java/io/iceberg/src/test/java/org/apache/beam/sdk/io/iceberg/IcebergIOLT.java
Outdated
Show resolved
Hide resolved
Looks like populating the data did have an effect. Pipelines used a lot more workers but finished in roughly the same amount of time. Write (2024-06-17_18_21_14-2727804973398206918)
Read (2024-06-17_19_20_46-7349681781857878664)
|
This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions. |
This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
Adding a load test for IcebergIO
Separating integration test (from #31220) into its own suite