yanghua commented on pull request #2443: URL: https://github.com/apache/hudi/pull/2443#issuecomment-785506762
> @yanghua : I synced up w/ Vinoth on this. here is our idea. we will make the config (naming) generic for now. but lets not make it wider to ignore exceptions for sync. bcoz, if sync explicitly fails probably user need to get notified. Since this diff was raised mainly for connect execption, we will leave it as is. But in future if some customer needs to ignore sync exceptions as well, we can consider silently ignoring those as well(so that we don't introduce a new config and can use the same config). wdyt? > If you agree, only fix we need for now is to fix the config naming. I can work on it and push in a small commit. OK, agree. The original intention of my idea is actually to not want the failure of synchronization to affect hudi's tasks. We have experienced situations where Hive is unavailable and all jobs fail. Then we completely ignored the failure, but when it failed, we implemented log warnings. But as a realization of the community, I agree with you. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
