Skip to content

Commit 8f1a107

Browse files
cleaned markers
1 parent 2bf20d8 commit 8f1a107

File tree

2 files changed

+3
-8
lines changed

2 files changed

+3
-8
lines changed

python/cocoindex/op.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -393,7 +393,7 @@ def enable_cache(self) -> bool:
393393
def behavior_version(self) -> int | None:
394394
return op_args.behavior_version
395395

396-
def timeout(self) -> int | None: # ✅ Add this
396+
def timeout(self) -> int | None:
397397
if op_args.timeout is not None:
398398
return op_args.timeout
399399

src/execution/evaluator.rs

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -372,7 +372,6 @@ async fn evaluate_op_scope(
372372
AnalyzedReactiveOp::Transform(op) => {
373373
let transform_key = format!("transform/{}{}", op_scope.scope_qualifier, op.name);
374374

375-
// Track transform operation start
376375
if let Some(ref op_stats) = operation_in_process_stats {
377376
op_stats.start_processing(&transform_key, 1);
378377
}
@@ -393,15 +392,14 @@ async fn evaluate_op_scope(
393392
let warn_handle = tokio::spawn(async move {
394393
tokio::time::sleep(warn_duration).await;
395394
eprintln!(
396-
"⚠️ WARNING: Function '{}' ({}) is taking longer than 30s",
395+
"WARNING: Function '{}' ({}) is taking longer than 30s",
397396
op_kind_for_warning, op_name_for_warning
398-
); // ✅ Show both
397+
);
399398
warn!(
400399
"Function '{}' ({}) is taking longer than 30s",
401400
op_kind_for_warning, op_name_for_warning
402401
);
403402
});
404-
// Execute with timeout
405403
let result = if op.function_exec_info.enable_cache {
406404
let output_value_cell = memory.get_cache_entry(
407405
|| {
@@ -420,7 +418,6 @@ async fn evaluate_op_scope(
420418
op.executor.evaluate(input_values)
421419
});
422420

423-
// Handle timeout
424421
let timeout_result = tokio::time::timeout(timeout_duration, eval_future).await;
425422
if timeout_result.is_err() {
426423
Err(anyhow!(
@@ -437,7 +434,6 @@ async fn evaluate_op_scope(
437434
} else {
438435
let eval_future = op.executor.evaluate(input_values);
439436

440-
// Handle timeout
441437
let timeout_result = tokio::time::timeout(timeout_duration, eval_future).await;
442438
if timeout_result.is_err() {
443439
Err(anyhow!(
@@ -455,7 +451,6 @@ async fn evaluate_op_scope(
455451

456452
warn_handle.abort();
457453

458-
// Track transform operation completion
459454
if let Some(ref op_stats) = operation_in_process_stats {
460455
op_stats.finish_processing(&transform_key, 1);
461456
}

0 commit comments

Comments
 (0)