Conversation
Contributor
Overall package sizeSelf size: 8.97 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 3.5.1 | 127.66 kB | 513.61 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 kB | | dc-polyfill | 0.1.11 | 25.74 kB | 25.74 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## watson/DEBUG-6081/bounded-queues #10155 +/- ##
===================================================================
Coverage ? 98.71%
===================================================================
Files ? 1025
Lines ? 160921
Branches ? 74
===================================================================
Hits ? 158860
Misses ? 2061
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 4, 2026 09:54
5b0dd17 to
dad9b24
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 4, 2026 10:24
dad9b24 to
8b7e432
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 4, 2026 12:02
8b7e432 to
8278842
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
2 times, most recently
from
September 4, 2026 15:41
2d0b3cc to
8f129ba
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 6, 2026 09:05
8f129ba to
017d35b
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 7, 2026 10:38
017d35b to
2f97506
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 11, 2026 11:21
2f97506 to
df96669
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 14, 2026 09:55
df96669 to
77ab918
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 14, 2026 10:55
77ab918 to
4dad4c7
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 14, 2026 12:57
4dad4c7 to
f5c28f6
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 15, 2026 07:44
f5c28f6 to
4e7da15
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 15, 2026 08:17
4e7da15 to
0bbdc67
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 15, 2026 10:27
0bbdc67 to
b03170d
Compare
watson
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 15, 2026 10:33
b03170d to
8df8a08
Compare
A probe whose condition threw was silently treated as not matching, so a broken condition (a typo in a variable name, a member access on an undefined value, ...) left the user with a probe that never fires and no explanation. The condition was also evaluated again, and threw again, on every single hit. Condition errors are now handled like the other tracers do it: the probe result carries the error in `evaluationErrors` (with the condition DSL as `expr` and the error as `message`) and nothing is captured, so the user can see why the probe is not producing regular results. Error results bypass the per-probe and global rate limits, and are instead limited by a per-probe throttle: after an error the condition is not evaluated again for five minutes, and one error result is reported per window. This also bounds the cost of a condition that is expensive to fail. The probe sampler records the error on the main thread and flags the sampled probe index, so the worker knows to fetch the error from the paused frame instead of capturing a snapshot.
BridgeAR
force-pushed
the
watson/DEBUG-6081/evaluation-errors
branch
from
September 15, 2026 12:23
8df8a08 to
53a3e54
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
A probe whose condition throws now emits an error-only result (
evaluationErrorswith the condition DSL and the error message, no captures) instead of being silently treated as "not matched". Error results bypass the rate limits and are throttled instead: after an error the condition is not evaluated again for five minutes and one error result is reported per window.Motivation
Users had no way to see why a probe with a typo in its condition never fired, and the failing condition was re-evaluated on every hit. Matches the .NET/Python behaviour and the
Test_Debugger_Runtime_Condition_Errorsystem tests.Additional Notes
The sampler records the error on the main thread and flags the sampled index (
CONDITION_ERROR_FLAG), so the worker fetches the error from the paused frame instead of capturing. Twodiagnostics.spec.jscases that used an undefined variable as a "false" condition were updated.