You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues-all@impala.apache.org by "Paul Rogers (JIRA)" <ji...@apache.org> on 2018/11/19 02:31:00 UTC

[jira] [Updated] (IMPALA-7868) Move UPDATE-only fields from ModifyStmt to UpdateStmt

     [ 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: issues-all-unsubscribe@impala.apache.org
For additional commands, e-mail: issues-all-help@impala.apache.org