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

NetworkTopology: getLeaf should check for invalid topologies

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.1.0, 2.0.0-alpha
    • None
    • None
    • 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

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

              Dates

                Created:
                Updated:
                Resolved: