Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-8159

NetworkTopology: getLeaf should check for invalid topologies

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.1.0, 2.0.0-alpha
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Currently, in NetworkTopology, getLeaf doesn't do too much validation on the InnerNode object itself. This results in us getting ClassCastException sometimes when the network topology is invalid. We should have a less confusing exception message for this case.

        Attachments

        1. HADOOP-8159.005.patch
          7 kB
          Colin McCabe
        2. HADOOP-8159.006.patch
          7 kB
          Colin McCabe
        3. HADOOP-8159.007.patch
          4 kB
          Colin McCabe
        4. HADOOP-8159.008.patch
          4 kB
          Colin McCabe
        5. HADOOP-8159.009.patch
          2 kB
          Colin McCabe
        6. HADOOP-8159-b1.005.patch
          6 kB
          Colin McCabe
        7. HADOOP-8159-b1.007.patch
          2 kB
          Colin McCabe

          Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. HADOOP-12173 NetworkTopology#add calls NetworkTopology#toString always

          • Major - Major loss of function.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. HADOOP-16662 Remove unnecessary InnerNode check in NetworkTopology#add()

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          relates to

          Test - A new unit, integration or system test. HDFS-3129 NetworkTopology: add test that getLeaf should check for invalid topologies

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

            Activity

              People

              • Assignee:
                cmccabe Colin McCabe
                Reporter:
                cmccabe Colin McCabe
              • Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: