Make QueryEngine code more readable by abstracting out debug and panic statements

Bunch of debug statements for latency tracking can be replaced by `tracing::debug_span!`

Bunch of panic statements should probably be replaced by returning an `Err` instead