My bad. Thank you for help me correct it.

2011/1/13 Edward J. Yoon <[email protected]>:
> Just FYI,
>
> In your patch, there was a simple bug, a kind of Type Mismatch Error.
>
> So, I've changed
>
> +            if(jip.getStatus().equals(JobStatus.SUCCEEDED)){
>
> to
>
> +          if (jip.getStatus().getRunState() == JobStatus.SUCCEEDED) {
>
>
> On Thu, Jan 13, 2011 at 1:22 PM, ChiaHung Lin (JIRA) <[email protected]> wrote:
>>
>>    [ 
>> https://issues.apache.org/jira/browse/HAMA-346?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12981126#action_12981126
>>  ]
>>
>> ChiaHung Lin commented on HAMA-346:
>> -----------------------------------
>>
>> I test the newly updated source checked out from svn, and it works well. 
>> Thanks for help arrange this.
>>
>>> Modify MniCluster so that developers can benefit when testing using Junit
>>> -------------------------------------------------------------------------
>>>
>>>                 Key: HAMA-346
>>>                 URL: https://issues.apache.org/jira/browse/HAMA-346
>>>             Project: Hama
>>>          Issue Type: Improvement
>>>          Components: test
>>>    Affects Versions: 0.2.0
>>>         Environment: Debian squeeze (kernel 2.6.3x) / JDK 1.6.x
>>>            Reporter: ChiaHung Lin
>>>            Assignee: ChiaHung Lin
>>>             Fix For: 0.2.0
>>>
>>>         Attachments: HAMA-346.patch, HAMA-346.patch, HAMA-346.patch, 
>>> HAMA-346_removeWarnings.patch
>>>
>>>
>>> The current junit test enviornment only provides default MiniCluster, which 
>>> does not launch any BSPMaster and GroomServer. Therefore, this patch is 
>>> provided for future junit testing when cluster environment is needed.
>>
>> --
>> This message is automatically generated by JIRA.
>> -
>> You can reply to this email to add a comment to the issue online.
>>
>>
>
>
>
> --
> Best Regards, Edward J. Yoon
> http://blog.udanax.org
> http://twitter.com/eddieyoon
>

Reply via email to