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

HIVE-5799 breaks backward compatibility for time values in config

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.14.0
    • Fix Version/s: 0.14.0
    • Component/s: Configuration
    • Labels:
      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

              • Assignee:
                alangates Alan Gates
                Reporter:
                alangates Alan Gates
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: