[
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]