-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
fix: Correct the update logic of timerecorder #34339
fix: Correct the update logic of timerecorder #34339
Conversation
Signed-off-by: bigsheeper <[email protected]>
…7-fix-timerecorder
@bigsheeper E2e jenkins job failed, comment |
/run-cpu-e2e |
@bigsheeper E2e jenkins job failed, comment |
@bigsheeper E2e jenkins job failed, comment |
rerun ut |
@bigsheeper E2e jenkins job failed, comment |
…7-fix-timerecorder
Signed-off-by: bigsheeper <[email protected]>
…7-fix-timerecorder
Signed-off-by: bigsheeper <[email protected]>
@bigsheeper E2e jenkins job failed, comment |
…7-fix-timerecorder
@bigsheeper E2e jenkins job failed, comment |
…7-fix-timerecorder
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #34339 +/- ##
==========================================
- Coverage 84.31% 80.86% -3.45%
==========================================
Files 825 1094 +269
Lines 113633 137794 +24161
==========================================
+ Hits 95809 111429 +15620
- Misses 13601 22134 +8533
- Partials 4223 4231 +8
|
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bigsheeper, czs007 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…date logic of ttchecker (#34383) This PR cherry-picks the following commits: - Try to improve cpu usage by refactoring the ttchecker logic and caching string. #33267 - Correct the update logic of timerecorder in the flowgraph to avoid false failure: "some node(s) haven't received input". #34339 issue: #33266, #34337 pr: #33267, #34339 --------- Signed-off-by: bigsheeper <[email protected]> Co-authored-by: Xiaofan <[email protected]>
Correct the update logic of timerecorder in the flowgraph to avoid false failure: "some node(s) haven't received input".
issue: #34337