Skip to content
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

Throttler: fix initial timestamp #1702

Merged

Conversation

jukzi
Copy link
Contributor

@jukzi jukzi commented Feb 20, 2024

Unlucky value range of System.nanoTime() could have prevented throttled Runnable to run.

Copy link
Contributor

github-actions bot commented Feb 20, 2024

Test Results

   918 files  +   918     918 suites  +918   47m 43s ⏱️ + 47m 43s
 7 452 tests + 7 452   7 301 ✅ + 7 301  151 💤 +151  0 ❌ ±0 
23 505 runs  +23 505  23 000 ✅ +23 000  505 💤 +505  0 ❌ ±0 

Results for commit 38131bf. ± Comparison against base commit 74289f5.

♻️ This comment has been updated with latest results.

@jukzi jukzi force-pushed the Throttler_fix_initial_timestamp branch from 7008257 to 51f0344 Compare March 5, 2024 07:10
Unlucky value range of System.nanoTime() could have prevented throttled
Runnable to run.
@jukzi jukzi force-pushed the Throttler_fix_initial_timestamp branch 3 times, most recently from c2c6bba to c47e2cc Compare March 5, 2024 11:01
@jukzi jukzi force-pushed the Throttler_fix_initial_timestamp branch from c47e2cc to 38131bf Compare March 5, 2024 11:28
@jukzi jukzi merged commit 12667ba into eclipse-platform:master Mar 5, 2024
16 checks passed
@jukzi jukzi deleted the Throttler_fix_initial_timestamp branch March 5, 2024 12:33
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.

3 participants