[jira] Commented: (PIG-1025) Should be able to set job priority through Pig Latin

["Ashutosh Chauhan (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/PIG-1025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12766771#action_12766771 ] 

Ashutosh Chauhan commented on PIG-1025:
---------------------------------------

Useful feature. Patch looks straightforward. In your test case you are only testing whether it parses it correctly or not, I will suggest to also test whether priority is actually set in the jobconf or not.

> Should be able to set job priority through Pig Latin
> ----------------------------------------------------
>
>                 Key: PIG-1025
>                 URL: https://issues.apache.org/jira/browse/PIG-1025
>             Project: Pig
>          Issue Type: New Feature
>          Components: grunt
>    Affects Versions: 0.4.0
>            Reporter: Kevin Weil
>            Priority: Minor
>             Fix For: 0.6.0
>
>         Attachments: PIG-1025.patch
>
>
> Currently users can set the job name through Pig Latin by saying
> set job.name 'my job name'
> The ability to set the priority would also be nice, and the patch should be small.  The goal is to be able to say
> set job.priority 'high'
> and throw a JobCreationException in the JobControlCompiler if the priority is not one of the allowed string values from the o.a.h.mapred.JobPriority enum: very_low, low, normal, high, very_high.   Case insensitivity makes this a little nicer.

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