[HIVE-8605] HIVE-5799 breaks backward compatibility for time values in config

• Type: Bug
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 0.14.0
• Fix Version/s: 0.14.0
• Component/s: Configuration
• Assignee: Alan Gates
• Reporter: Alan Gates
• Created: 25/Oct/14 21:14
• Updated: 01/Oct/19 22:07
• Resolved: 28/Oct/14 20:43

Description

It is legal for long values in the config file to have an L or for float values to have an f. For example, the default value for hive.compactor.check.interval was 300L. As part of HIVE-5799, many long values were converted to TimeUnit. Attempts to read these values now throw "java.lang.IllegalArgumentException: Invalid time unit l"

We need to change this to ignore the L or f, so that users existing config files don't break. I propose to do this by changing HiveConf.unitFor to detect the L or f and interpret it to mean the default time unit.

Issue Links

• relates to: HIVE-5799 session/operation timeout for hiveserver2