[
https://issues.apache.org/jira/browse/PIG-1333?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12876111#action_12876111
]
Dmitriy V. Ryaboy commented on PIG-1333:
----------------------------------------
That's a heck of a patch. I am really looking forward to having this available.
Not sure you need the map in the PIG_FEATURE enum. You can get an enum by
offset using PIG_FEATURE.values(), so no need for the constructor; you can get
a string representation using pigFeature.name() or pigFeature.toString(), so no
need for getString(); and you can get the ordinal using pigFeature.ordinal().
Granted, the ordinals are 0-based, but you can just throw in an dummy value for
the 0th spot to preserve all the offsets as they are.
I see that you explicitly pull out the known and enumerated Pig counters. Any
reason not to make all other job counters available as well via the same
interface?
> API interface to Pig
> --------------------
>
> Key: PIG-1333
> URL: https://issues.apache.org/jira/browse/PIG-1333
> Project: Pig
> Issue Type: Improvement
> Reporter: Olga Natkovich
> Assignee: Richard Ding
> Fix For: 0.8.0
>
> Attachments: PIG-1333.patch
>
>
> It would be nice to make Pig more friendly for applications like workflow
> that would be executing pig scripts on user behalf.
> Currently, they would have to use pig command line to execute the code;
> however, this has limitation on the kind of output that would be delivered.
> For instance, it is hard to produce error information that is easy to use
> programatically or collect statistics.
> The proposal is to create a class that mimics the behavior of the Main but
> gives users a status object back. The the main code of pig would look
> somethig like:
> public static void main(String args[])
> {
> PigStatus ps = PigMain.exec(args);
> exit (PigStatus.rc);
> }
> We need to define the following:
> - Content of PigStatus. It should at least include
> * return code
> * error string
> * exception
> * statistics
> - A way to propagate the status class through pig code
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
