[HIVE-19379] Docs Are Incorrect for "hive.lock.sleep.between.retries"

• Type: Improvement
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 2.4.0, 3.0.0
• Component/s: Documentation
• Assignee: Unassigned
• Reporter: David Mollitor
• Created: 01/May/18 19:54
• Updated: 29/May/18 20:47

Description

https://cwiki.apache.org/confluence/display/Hive/Configuration+Properties#ConfigurationProperties-Locking

Default Value: 60
Added In: Hive 0.7.0 with HIVE-1293
The sleep time (in seconds) between various retries.

Actually, it's the sleep time (don't specify a unit) and the default value is 60s. If no unit is specified, it would be 60 milliseconds.

ZooKeeperHiveLockManager.java

  public void refresh() {
    HiveConf conf = ctx.getConf();
    sleepTime = conf.getTimeVar(
        HiveConf.ConfVars.HIVE_LOCK_SLEEP_BETWEEN_RETRIES, TimeUnit.MILLISECONDS);
    numRetriesForLock = conf.getIntVar(HiveConf.ConfVars.HIVE_LOCK_NUMRETRIES);
    numRetriesForUnLock = conf.getIntVar(HiveConf.ConfVars.HIVE_UNLOCK_NUMRETRIES);
  }