Uploaded image for project: 'Hadoop YARN'
  1. Hadoop YARN
  2. YARN-10894

Follow up YARN-10237: fix the new test case in TestRMWebServicesCapacitySched

    XMLWordPrintableJSON

Details

    • Reviewed

    Description

      The new test case (added with YARN-10237) in TestRMWebServicesCapacitySched is not testing anything:

      A new queue "C" was created:

      final String C = CapacitySchedulerConfiguration.ROOT + ".c";    config.setCapacity(C, "[memory=1024]");
      

      but it was not added to the top level queues:

      // Define top-level queues
      config.setQueues(CapacitySchedulerConfiguration.ROOT,
          new String[] {"a", "b"});

      So it does not test anything.

      The asserts later should check the queue name like this:

      - if (q.equals("c")) { 
      + if (q.equals("root.c")) { 

      Unfortunately some other asserts fail, so either the test is faulty or there is some issue with the code.

      Attachments

        Activity

          People

            tdomok Tamas Domok
            tdomok Tamas Domok
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 1h 40m
                1h 40m