[ https://issues.apache.org/jira/browse/PIG-990?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12782697#action_12782697 ]
Dmitriy V. Ryaboy commented on PIG-990: --------------------------------------- This patch lost a parenthesis somewhere in my merge attempt. I am running the fixed version through test-commit and findbugs. Ashutosh: pinOptions is userPinned. If we want to add systemPinned, we can do that as a separate patch (we don't even need that for the cbo as it stands, right?). > 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 > Assignee: Dmitriy V. Ryaboy > Priority: Minor > Fix For: 0.7.0 > > Attachments: pinned_options.patch, pinned_options_2.patch, > pinned_options_3.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.