You are viewing a plain text version of this content. The canonical link for it is here.

Posted to derby-dev@db.apache.org by "Kristian Waagan (JIRA)" <ji...@apache.org> on 2010/11/12 12:31:18 UTC

[jira] Resolved: (DERBY-4899) Refactor access to primary structures in AlterTableConstantAction

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

Kristian Waagan resolved DERBY-4899.
------------------------------------

          Resolution: Fixed
       Fix Version/s: 10.7.1.0
    Issue & fix info:   (was: [Patch Available])

Committed patch 1b to trunk with revision 1034347.

> Refactor access to primary structures in AlterTableConstantAction
> -----------------------------------------------------------------
>
>                 Key: DERBY-4899
>                 URL: https://issues.apache.org/jira/browse/DERBY-4899
>             Project: Derby
>          Issue Type: Improvement
>          Components: SQL
>    Affects Versions: 10.7.1.0
>            Reporter: Kristian Waagan
>            Assignee: Kristian Waagan
>            Priority: Minor
>             Fix For: 10.7.1.0
>
>         Attachments: derby-4899-1a-cleanup.diff, derby-4899-1b-cleanup.diff
>
>
> Many of the private methods in impl.sql.execute.AlterTableConstantAction take the activation (and sometimes a number of references obtained through it) as an argument. This seems unnecessary, and it clutters the code with argument passing, boilerplate code and variable hiding.

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