Skip to content

Use nextWithTimer() instead of next() in Operator#15071

Merged
JackieTien97 merged 1 commit intomasterfrom
operatorNext
Mar 12, 2025
Merged

Use nextWithTimer() instead of next() in Operator#15071
JackieTien97 merged 1 commit intomasterfrom
operatorNext

Conversation

@lancelly
Copy link
Contributor

No description provided.

@lancelly lancelly changed the title Use nextWithTime() instead of next() in Operator Use nextWithTimer() instead of next() in Operator Mar 12, 2025
@sonarqubecloud
Copy link

@JackieTien97 JackieTien97 merged commit 40dd031 into master Mar 12, 2025
19 checks passed
@JackieTien97 JackieTien97 deleted the operatorNext branch March 12, 2025 10:04
@codecov
Copy link

codecov bot commented Mar 12, 2025

Codecov Report

Attention: Patch coverage is 25.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 39.43%. Comparing base (a7ebfa8) to head (52cf2d3).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...ution/operator/process/AssignUniqueIdOperator.java 0.00% 1 Missing ⚠️
...ion/operator/process/EnforceSingleRowOperator.java 0.00% 1 Missing ⚠️
...execution/operator/sink/ShuffleHelperOperator.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master   #15071   +/-   ##
=========================================
  Coverage     39.43%   39.43%           
  Complexity      193      193           
=========================================
  Files          4596     4596           
  Lines        295115   295115           
  Branches      36688    36688           
=========================================
+ Hits         116385   116390    +5     
+ Misses       178730   178725    -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

JackieTien97 pushed a commit that referenced this pull request Apr 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants