hadoop-mapreduce-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "rahul k singh (JIRA)" <j...@apache.org>
Subject [jira] Commented: (MAPREDUCE-1067) Default state of queues is undefined when unspecified
Date Wed, 09 Dec 2009 09:02:18 GMT

    [ https://issues.apache.org/jira/browse/MAPREDUCE-1067?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12787985#action_12787985
] 

rahul k singh commented on MAPREDUCE-1067:
------------------------------------------

Queue.java
1.we can use children instead of this.children , similary for this.name.

QueueState.java
1. comment in getState is incorrect. 
// check if state is in uppercase and then set the state.
instead of above it should be  
//we are passing uppercase , as java returns the respective case enum object 
//so for "stopped" valueOf returns the lowercase enum "stopped" object.

2. TestQueueManager.
  There is validation check only for child queue and acls. we need to also include a
  state check . That is validation for case where in we have state and child queue defined.

> Default state of queues is undefined when unspecified
> -----------------------------------------------------
>
>                 Key: MAPREDUCE-1067
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1067
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: jobtracker
>    Affects Versions: 0.21.0
>            Reporter: V.V.Chaitanya Krishna
>            Assignee: V.V.Chaitanya Krishna
>            Priority: Blocker
>             Fix For: 0.21.0
>
>         Attachments: MAPREDUCE-1067-1.patch, MAPREDUCE-1067-2.patch, MAPREDUCE-1067-3.patch,
MAPREDUCE-1067-4.patch
>
>
> Currently, if the state of a queue is not specified, it is being set to "undefined" state
instead of running state.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message