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

Paul Rogers updated IMPALA-7868:
--------------------------------
    Description: 
A subset of the FE parse node hierarchy:

{noformat}
ParseNode
|- StatementBase
   |- ModifyStmt
      |- DeleteStmt
      |- UpdateStmt
{noformat}

The {{ModifyStmt}} holds a number of members that are used only by the 
{{UpdateStmt}}, but which make no sense for the {{DeleteStmt}}.

There are some tricky bits because of columns referenced in the {{WHERE}} 
clause.

This ticket asks to rationalize the class structure, moving {{UPDATE}}-only 
fields to the {{UpdateStmt}} class.

 

  was:
A subset of the FE parse node hierarchy:

{noformat}
ParseNode
|- StatementBase
   |- ModifyStmt
      |- DeleteStmt
      |- UpdateStmt
{noformat}

The {{ModifyStmt}} holds a number of members that are used only by the 
{{UpdateStmt}}, but which make no sense fr the {{DeleteStmt}}.

This ticket asks to rationalize the class structure, moving {{UPDATE}}-only 
fields to the {{UpdateStmt}} class.


> Move UPDATE-only fields from ModifyStmt to UpdateStmt
> -----------------------------------------------------
>
>                 Key: IMPALA-7868
>                 URL: https://issues.apache.org/jira/browse/IMPALA-7868
>             Project: IMPALA
>          Issue Type: Improvement
>          Components: Frontend
>    Affects Versions: Impala 3.0
>            Reporter: Paul Rogers
>            Assignee: Paul Rogers
>            Priority: Minor
>
> A subset of the FE parse node hierarchy:
> {noformat}
> ParseNode
> |- StatementBase
>    |- ModifyStmt
>       |- DeleteStmt
>       |- UpdateStmt
> {noformat}
> The {{ModifyStmt}} holds a number of members that are used only by the 
> {{UpdateStmt}}, but which make no sense for the {{DeleteStmt}}.
> There are some tricky bits because of columns referenced in the {{WHERE}} 
> clause.
> This ticket asks to rationalize the class structure, moving {{UPDATE}}-only 
> fields to the {{UpdateStmt}} class.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to