[ 
https://issues.apache.org/jira/browse/PIG-18?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Craig Macdonald updated PIG-18:
-------------------------------

    Attachment: PIG-18.patch-v3craigm-10032008

Here's an updated version of my earlier patch. This even compiles, and I tested 
it ;-)

All ssh code has been removed. Consequently the hod.server property has been 
removed.

Olga, I couldn't follow your HOD commands versus the HOD documentation 
http://hadoop.apache.org/core/docs/r0.16.0/hod.html, so I recoded it to follow 
that document. As related issue is them that the command line should be passed 
as an array, to ensure that spaces within the double quoted region are not 
tokenised by Runtime.exec(String).

I added a System property simply named "hod", which takes the value of $hodRoot 
from pig.pl. This is what defines if HOD should be used, and also allows HOD to 
be explicitly specified, not just in the user's path. 

The ExecException "Failed to connect to HOD. Return code" was never actually 
thrown (the throw was missing). doHod() now propagates any ExecException thrown 
in doHod(), rather than wrapping it in another one.

A future "feature"  for debugging log level would be to capture the HOD 
standard output and log that.

Craig

> Making Pig work with HOD 0.4 and hadoop 0.16
> --------------------------------------------
>
>                 Key: PIG-18
>                 URL: https://issues.apache.org/jira/browse/PIG-18
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>            Reporter: Olga Natkovich
>            Assignee: Olga Natkovich
>            Priority: Critical
>         Attachments: hadoop16.jar, PIG-18.patch, 
> PIG-18.patch-v2craigm-07032008, PIG-18.patch-v3craigm-10032008
>
>
> Pig needs to work with the soon to be released hadoop on demand (HOD) version.

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

Reply via email to