Uploaded image for project: 'Hive'
  1. Hive
  2. HIVE-8605

HIVE-5799 breaks backward compatibility for time values in config

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • 0.14.0
    • 0.14.0
    • Configuration
    • None

    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.

      Attachments

        1. HIVE-8605.2.patch
          3 kB
          Alan Gates
        2. HIVE-8605.patch
          3 kB
          Alan Gates

        Issue Links

          Activity

            People

              gates Alan Gates
              gates Alan Gates
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: