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

NetworkTopology: getLeaf should check for invalid topologies

          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-b1.005.patch
                6 kB
                Colin P. McCabe
              2. HADOOP-8159.005.patch
                7 kB
                Colin P. McCabe
              3. HADOOP-8159.006.patch
                7 kB
                Colin P. McCabe
              4. HADOOP-8159.007.patch
                4 kB
                Colin P. McCabe
              5. HADOOP-8159.008.patch
                4 kB
                Colin P. McCabe
              6. HADOOP-8159.009.patch
                2 kB
                Colin P. McCabe
              7. HADOOP-8159-b1.007.patch
                2 kB
                Colin P. 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
                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 P. McCabe
                      Reporter:
                      cmccabe Colin P. McCabe
                    • Votes:
                      0 Vote for this issue
                      Watchers:
                      4 Start watching this issue

                      Dates

                      • Created:
                        Updated:
                        Resolved: