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

HIVE-5799 breaks backward compatibility for time values in config

Log workAgile BoardRank to TopRank to BottomVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    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

          $i18n.getText('security.level.explanation', $currentSelection) Viewable by All Users
          Cancel

            People

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment