[jira] Commented: (PIG-990) Provide a way to pin LogicalOperator Options

["Alan Gates (JIRA)" <ji...@apache.org>]

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

Alan Gates commented on PIG-990:
--------------------------------

Looks good.

One comment, rather than referring to the default join as "regular" I think we should refer to it as "hash" or "symmetric hash", since these accurately describe how it works.  That way users can specify the type of join they want, and if for whatever reason we switch the default they'll still get what they want.

> Provide a way to pin LogicalOperator Options
> --------------------------------------------
>
>                 Key: PIG-990
>                 URL: https://issues.apache.org/jira/browse/PIG-990
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>            Reporter: Dmitriy V. Ryaboy
>            Priority: Minor
>             Fix For: 0.6.0
>
>         Attachments: pinned_options.patch
>
>
> This is a proactive patch, setting up the groundwork for adding an optimizer.
> Some of the LogicalOperators have options. For example, LOJoin has a variety of join types (regular, fr, skewed, merge), which can be set by the user or chosen by a hypothetical optimizer.  If a user selects a join type, pig philoophy guides us to always respect the user's choice and not explore alternatives.  Therefore, we need a way to "pin" options.  

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