nsivabalan commented on PR #11580: URL: https://github.com/apache/hudi/pull/11580#issuecomment-2368683878
hey @codope : that would be tough to validate. bcoz, for some of the callers, lock is taking 3 to 4 call stack above and don't feel its worth passing txnManager all the way to validate that lock is acquired. I do get your intent. But for mdt writes also, we ensure we call it from within a lock. We don't have ``` ValidationUtils.checkArgument(txnManager.isLockRequired(),"Lock needs to be acquired in order to perform timestamp validation"); ``` this sort of validation for every write to MDT. -- 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]
