You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2022/02/08 03:31:35 UTC

[GitHub] [hudi] danny0405 commented on pull request #3887: [HUDI-2648] Retry FileSystem action instead of failed directly.

danny0405 commented on pull request #3887:
URL: https://github.com/apache/hudi/pull/3887#issuecomment-1032179002


   > @zhangyue19921010 : lets not wait for Vinoth. he has been busy recently. Lets go ahead for now. can we do this. add a property named supportsRetry or something to each StorageScheme. So, only for those storage schemes where retries are supported, we can check the config (fileSystemRetryConfig.isFileSystemActionRetryEnable()) and then invoke our retry Helper. for eg, in S3 this can be disabled.
   
   One thing we should be very careful with is in what exception/error we should do a retry here ? If it is a normal user triggered exception, should we still retry here ?


-- 
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: commits-unsubscribe@hudi.apache.org

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