nsivabalan commented on pull request #2443: URL: https://github.com/apache/hudi/pull/2443#issuecomment-784233741
@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. 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. 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. ---------------------------------------------------------------- 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]
