Skip to content

Commit fe63aad

Browse files
authored
[BUG] wal3 Sometimes returns an "Internal" when it should return "LogContentionFailure" (#5916)
## Description of changes Because of how work gets thrown away during contention, it was possible for a channel to close and generate an internal error. This casts that condition to be more descriptive. ## Test plan CI ## Migration plan N/A ## Observability plan N/A ## Documentation Changes N/A
1 parent 6eec387 commit fe63aad

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

rust/wal3/src/manifest_manager.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -353,7 +353,7 @@ impl ManifestManager {
353353
match rx.await {
354354
Ok(None) => Ok(()),
355355
Ok(Some(err)) => Err(err),
356-
Err(_) => Err(Error::internal(file!(), line!())),
356+
Err(_) => Err(Error::LogContentionFailure),
357357
}
358358
}
359359

0 commit comments

Comments
 (0)