[GitHub] [hudi] xushiyan commented on pull request #8508: [MINOR][DOCS] Update docs for fs based lock provider

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

xushiyan commented on PR #8508:
URL: https://github.com/apache/hudi/pull/8508#issuecomment-1515822333

   > Lets check w/ the authors of this feature (lock provider) to clarify if they got a chance to test it in cloud stores. bcoz, I do see that we write some current writer info to the file. So, wondering if they meant to deduce if file creation succeeded (incidentally by 2 concurrent writers), can we use the contents of the file to verify that current owner is the one which is actually owning. if not, fail the lock.
   > 
   > not sure if thats what their intention was. but we can clarify.
   
   the guard rail was added in https://github.com/apache/hudi/pull/7614


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