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