lintingbin opened a new issue, #3384:
URL: https://github.com/apache/amoro/issues/3384

   ### What happened?
   
   In our production environment, we often encounter commit failure errors in 
Amoro. I tried to change the commit.retry.num-retries configuration on the 
catalog in Amoro to 30, but found that it didn't take effect. After 
troubleshooting, I discovered that during Amoro commit, the 
commitSimpleTransaction function in the org.apache.iceberg.BaseTransaction 
class is executed. This function retrieves the commit.retry.num-retries 
parameter directly from the metadata file's content, so the configuration set 
on the catalog does not take effect.
   
   ### Affects Versions
   
   0.7.1
   
   ### What table formats are you seeing the problem on?
   
   Iceberg
   
   ### What engines are you seeing the problem on?
   
   AMS
   
   ### How to reproduce
   
   _No response_
   
   ### Relevant log output
   
   _No response_
   
   ### Anything else
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's Code of Conduct


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to