Skip to content

Commit aabf016

Browse files
authored
Bump rejit timeout to 200ms (#7301)
## Summary of changes Bump the ReJIT timeout from 100ms to 200ms ## Reason for change In #7289 we bumped it from 100ms to 150ms as a bandaid on timeouts we started seeing in CI (which we believe are environment related). We're still seeing timeouts though, so bumping it further ## Implementation details `150ms`->`200ms` ## Test coverage This is the test, hopefully no more timeouts ## Other details This is all a little concerning, but is still preferable to ignoring the timeouts. We have considered auto-retries in CI, but the fact that it can happen to customers potentially too means that we think bumping the timeout is the better option at the moment.
1 parent 0b3b442 commit aabf016

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tracer/src/Datadog.Tracer.Native/cor_profiler.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -695,7 +695,7 @@ HRESULT STDMETHODCALLTYPE CorProfiler::ModuleLoadFinished(ModuleID module_id, HR
695695
promise);
696696

697697
// wait and get the value from the future<ULONG>
698-
const auto status = future.wait_for(150ms);
698+
const auto status = future.wait_for(200ms);
699699

700700
if (status != std::future_status::timeout)
701701
{
@@ -1314,7 +1314,7 @@ HRESULT CorProfiler::TryRejitModule(ModuleID module_id, std::vector<ModuleID>& m
13141314
promise);
13151315

13161316
// wait and get the value from the future<ULONG>
1317-
const auto status = future.wait_for(150ms);
1317+
const auto status = future.wait_for(200ms);
13181318

13191319
if (status != std::future_status::timeout)
13201320
{

0 commit comments

Comments
 (0)