Skip to content

Commit

Permalink
Temporarily replace client tracing for GetNextUnknown with fallback t…
Browse files Browse the repository at this point in the history
…o FetchNext
  • Loading branch information
mappum committed Oct 15, 2024
1 parent 73a3a56 commit b7259fd
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/client/exec.rs
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,11 @@ where
(key.clone(), StepResult::FetchKey(key))
}
Err(Error::StoreErr(store::Error::GetNextUnknown(key))) => {
(key.clone(), StepResult::FetchNext(key))
// (key.clone(), StepResult::FetchNext(key))
// TODO: optimistically attempt to trace and only use fetchnext as fallback. we
// only do this because unwrapping a collections::Map::Iter entry
// does not push a trace yet
return Ok(StepResult::FetchNext(key));
}
Err(Error::StoreErr(store::Error::GetPrevUnknown(maybe_key))) => {
if let Some(key) = maybe_key {
Expand Down

0 comments on commit b7259fd

Please sign in to comment.