From bae5f553a295d6c4532a5454ff6ca78e2c63664e Mon Sep 17 00:00:00 2001 From: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com> Date: Thu, 3 Jun 2021 00:58:29 +0000 Subject: [PATCH] Expand docs docs for RetryOptions (#2385) * Better docs for RetryOptions * Update sdk/core/azure-core/inc/azure/core/http/policies/policy.hpp Co-authored-by: Ahson Khan Co-authored-by: Anton Kolesnyk Co-authored-by: Ahson Khan --- .../inc/azure/core/http/policies/policy.hpp | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/sdk/core/azure-core/inc/azure/core/http/policies/policy.hpp b/sdk/core/azure-core/inc/azure/core/http/policies/policy.hpp index 611e494df..a4dafd081 100644 --- a/sdk/core/azure-core/inc/azure/core/http/policies/policy.hpp +++ b/sdk/core/azure-core/inc/azure/core/http/policies/policy.hpp @@ -59,34 +59,35 @@ namespace Azure { namespace Core { namespace Http { namespace Policies { }; /** - * @brief HTTP request retry options. + * @brief The set of options that can be specified to influence how retry attempts are made, and a + * failure is eligible to be retried. * @note See https://azure.github.io/azure-sdk/general_azurecore.html#retry-policy. * */ struct RetryOptions final { /** - * @brief Maximum number of attempts to retry. + * @brief The maximum number of retry attempts before giving up. * */ int32_t MaxRetries = 3; /** - * @brief Mimimum amount of milliseconds between retry attempts. + * @brief The minimum permissible delay between retry attempts. * @note See https://en.cppreference.com/w/cpp/chrono/duration. * */ std::chrono::milliseconds RetryDelay = std::chrono::seconds(4); /** - * @brief Mimimum amount of milliseconds between retry attempts. + * @brief The maximum permissible delay between retry attempts. * @note See https://en.cppreference.com/w/cpp/chrono/duration. * */ std::chrono::milliseconds MaxRetryDelay = std::chrono::minutes(2); /** - * @brief HTTP status codes to retry on. + * @brief The HTTP status codes that indicate when an operation should be retried. * */ std::set StatusCodes{