[I] Make the return value of LockProvider's tryLock method more reasonable [hudi]

["hudi-bot (via GitHub)" <gi...@apache.org>]

hudi-bot opened a new issue, #15283:
URL: https://github.com/apache/hudi/issues/15283

   For now, zk lock provider and hive lock provider never return false when tryLock, hence it will always return an Exception, Why can't it return false? is it make sense to always return Exception? I feel maybe we need to make sure the behavior is properly first
   if want to return false or return HoodieLockException or return InterruptedException, should mean something
   eg: if lockprovider returns a IllegalArgumentException we don't need to retry in LockManager since it will always illegal
   
   ## JIRA info
   
   - Link: https://issues.apache.org/jira/browse/HUDI-4413
   - Type: Improvement


-- 
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.apache.org

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

Re: [I] Make the return value of LockProvider's tryLock method more reasonable [hudi]

["rangareddy (via GitHub)" <gi...@apache.org>]

rangareddy commented on issue #15283:
URL: https://github.com/apache/hudi/issues/15283#issuecomment-5165418668

   CC @yihua @nsivabalan 


-- 
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