gaborkaszab opened a new pull request, #10995:
URL: https://github.com/apache/iceberg/pull/10995
There is a comment for FastAppend that mentions the number of commit retries
done by the class. I removed it for 2 reasons:
1) Not the FastAppend class does the commit so the commit retries
shouldn't be commented there.
2) The number of commit retries is configurable via a table property so
mentioning a hard-coded
value in the comment is incorrect.
Added a new test that adjusts the COMMIT_NUM_RETRIES table property.
Adjusted existing tests to exercise the boundaries of the number of commit
retries.
--
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
