Skip to content

Commit f297a01

Browse files
committed
re-enable for test
1 parent af5a43a commit f297a01

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

core/services/llo/observation/observation_context.go

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -339,14 +339,22 @@ func (oc *observationContext) run(ctx context.Context, streamID streams.StreamID
339339
ex, isExecuting := oc.executions[p]
340340
if isExecuting {
341341
oc.executionsMu.Unlock()
342+
// wait for it to finish
343+
select {
344+
case <-ex.done:
345+
return ex.run, ex.trrs, ex.err
346+
case <-ctx.Done():
347+
return nil, nil, ctx.Err()
348+
}
342349
// Block unconditionally until the executing goroutine finishes.
343350
// All waiters must receive the same (run, trrs, err) tuple to
344351
// prevent partial cache writes for streams sharing a pipeline.
345352
// This is safe because p.Run(ctx) respects context cancellation:
346353
// when ctx expires, HTTP requests abort, the scheduler collects
347354
// all error results, and p.Run returns promptly.
348-
<-ex.done
349-
return ex.run, ex.trrs, ex.err
355+
// RE-ENABLE after test
356+
// <-ex.done
357+
// return ex.run, ex.trrs, ex.err
350358
}
351359

352360
// execute here

0 commit comments

Comments
 (0)