massdosage commented on a change in pull request #2893:
URL: https://github.com/apache/iceberg/pull/2893#discussion_r687764150



##########
File path: site/docs/configuration.md
##########
@@ -91,7 +91,7 @@ Iceberg tables support table properties to configure table 
behavior, like the de
 | commit.retry.num-retries           | 4                | Number of times to 
retry a commit before failing              |
 | commit.retry.min-wait-ms           | 100              | Minimum time in 
milliseconds to wait before retrying a commit |
 | commit.retry.max-wait-ms           | 60000 (1 min)    | Maximum time in 
milliseconds to wait before retrying a commit |
-| commit.retry.total-timeout-ms      | 1800000 (30 min) | Maximum time in 
milliseconds to wait before retrying a commit |
+| commit.retry.total-timeout-ms      | 1800000 (30 min) | Total timeout in 
milliseconds between retrying a commit.                              |

Review comment:
       ```suggestion
   | commit.retry.total-timeout-ms      | 1800000 (30 min) | Total timeout in 
milliseconds when retrying a commit.                              |
   ```
   I think this is a better description of what this does?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to