Hadoop Common
  1. Hadoop Common
  2. HADOOP-2081

Configuration getInt, getLong, and getFloat replace invalid numbers with the default value

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.22.0
    • Fix Version/s: 0.23.0
    • Component/s: conf
    • Labels:
      None
    • Hadoop Flags:
      Incompatible change, Reviewed
    • Release Note:
      Invalid configuration values now result in a number format exception rather than the default value being used.

      Description

      The current behavior of silently replace invalid numbers in the configuration with the default value leads to hard to diagnose problems. The methods should throw an exception to signal that the input was invalid.

      1. HADOOP-2081.r1.diff
        5 kB
        Harsh J
      2. HADOOP-2081.r2.diff
        7 kB
        Harsh J
      3. HADOOP-2081.r3.diff
        7 kB
        Harsh J

        Activity

        Owen O'Malley created issue -
        Nigel Daley made changes -
        Field Original Value New Value
        Fix Version/s 0.16.0 [ 12312740 ]
        Harsh J made changes -
        Assignee Owen O'Malley [ owen.omalley ] Harsh J [ qwertymaniac ]
        Harsh J made changes -
        Attachment HADOOP-2081.r1.diff [ 12486745 ]
        Harsh J made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hadoop Flags [Incompatible change]
        Affects Version/s 0.22.0 [ 12314296 ]
        Fix Version/s 0.23.0 [ 12315569 ]
        Harsh J made changes -
        Attachment HADOOP-2081.r2.diff [ 12486746 ]
        Harsh J made changes -
        Attachment HADOOP-2081.r3.diff [ 12486773 ]
        Eli Collins made changes -
        Hadoop Flags [Incompatible change] [Incompatible change, Reviewed]
        Release Note Invalid configuration values now result in a number format exception rather than the default value being used.
        Eli Collins made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Arun C Murthy made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Harsh J
            Reporter:
            Owen O'Malley
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development