Thanks for reviewing this, Arun.
Here are my responses to your review notes:
> 1.Heading should be 'FifoScheduler' rather than 'default queue'
The "Default Scheduler" heading came from the RmController class, which I did not change.
However, I did change the QueuesBlock header to be 'FifoScheduler Queue'.
I also left "'default' Queue Status" because 'default' is coming from the getQueueName() metohod.
Please take a look at the new screenshot to let me know what you think.
> 2.No need to show a bar for queue
Hmm... I think Ramya specifically wanted the queue bar. Can you please check with her?
Ramya, what do you think about this one?
> 3.Alignment of text is broken (should be centre)
>4.Queue State is 'unknown' -> should be RUNNING/STOPPED
This value is coming from getQueueState() method. It was 'unknown' because the FifoScheduler() constructor was not initializing the queue state. I added a fix to this in FifoScheduler.java and it is working now.
> 5.Showing queue capacity is not useful with a single queue i.e drop max/total/current capacity
> Also, is there some unit tests we can add?
I updated the following test: