refactor: log server error during object store retries (#5294)

Similar to transport errors, log the server error as well. That should help debugging.
apache · Jan 11, 2024 · fcfe36f · fcfe36f
1 parent db81108
commit fcfe36f
Showing 1 changed file with 14 additions and 2 deletions.
diff --git a/object_store/src/client/retry.rs b/object_store/src/client/retry.rs
@@ -246,7 +246,13 @@ impl RetryExt for reqwest::RequestBuilder {
 
                             let sleep = backoff.next();
                             retries += 1;
-                            info!("Encountered server error, backing off for {} seconds, retry {} of {}", sleep.as_secs_f32(), retries, max_retries);
+                            info!(
+                                "Encountered server error, backing off for {} seconds, retry {} of {}: {}",
+                                sleep.as_secs_f32(),
+                                retries,
+                                max_retries,
+                                e,
+                            );
                             tokio::time::sleep(sleep).await;
                         }
                     },
@@ -277,7 +283,13 @@ impl RetryExt for reqwest::RequestBuilder {
                         }
                         let sleep = backoff.next();
                         retries += 1;
-                        info!("Encountered transport error ({}) backing off for {} seconds, retry {} of {}", e, sleep.as_secs_f32(), retries, max_retries);
+                        info!(
+                            "Encountered transport error backing off for {} seconds, retry {} of {}: {}", 
+                            sleep.as_secs_f32(),
+                            retries,
+                            max_retries,
+                            e,
+                        );
                         tokio::time::sleep(sleep).await;
                     }
                 }