Prefer to return `Result<(), Error>` over `Option<Error>`

[dbolduc]

Returning a Result is more idiomatic. Results are easier to chain with ?.

We should change

• PollingErrorPolicy::on_in_progress()
• RetryPolicy::on_throttle()

Note that these fns are hidden from our docs, and thus not part of our public API.