pvary commented on issue #2301:
URL: https://github.com/apache/iceberg/issues/2301#issuecomment-939903825


   @zhangminglei:
   The code path protected by the lock does this:
   ```
   acquireLock()
     loadHmsTable()
     setHmsTableParameters(newMetadataLocation, tbl, metadata.properties(), 
removedProps, hiveEngineEnabled, summary);
     persistTable(tbl, updateHiveTable)
   unlock(lockId)
   ```
   
   This is 2 HMS calls and some object data update. So this should not take 
very long. Also the TXN_TIMEOUT is configured to 300s by default.
   
   All that said, you are right, that in extreme scenarios this could be a 
problem.


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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to