-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[BEAM-14439] [BEAM-12673] Add extra details to PubSub matcher errors #17586
Conversation
Can one of the admins verify this patch? |
1 similar comment
Can one of the admins verify this patch? |
Codecov Report
@@ Coverage Diff @@
## master #17586 +/- ##
==========================================
+ Coverage 73.83% 73.91% +0.08%
==========================================
Files 691 692 +1
Lines 91252 91598 +346
==========================================
+ Hits 67373 67706 +333
- Misses 22647 22660 +13
Partials 1232 1232
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
R: @tvalentyn |
Thanks. You can also mention something on relevant bugs that we are looking forward to seeing this error again next time it fails. |
Hi, thanks for the commit. I found a little thing. My old code: and
The error: My fix: |
Sorry that this broke your old code! Thanks for the update, and glad to know that you're able to make an adjustment to fix it. |
Adding more details to the error message to
PubSubMessageMatcher
when the actual and expected messages don't match. This will aid in future debugging when test jobs fail.Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
R: @username
).[BEAM-XXX] Fixes bug in ApproximateQuantiles
, where you replaceBEAM-XXX
with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.CHANGES.md
with noteworthy changes.See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)
See CI.md for more information about GitHub Actions CI.