[ http://issues.apache.org/jira/browse/TUSCANY-483?page=all ]

Brent Daniel reassigned TUSCANY-483:
------------------------------------

    Assign To: Brent Daniel

> Some refactoring is necessary after adding the DAS/CommandGroup
> ---------------------------------------------------------------
>
>          Key: TUSCANY-483
>          URL: http://issues.apache.org/jira/browse/TUSCANY-483
>      Project: Tuscany
>         Type: Improvement

>   Components: Java DAS RDB
>     Versions: Java-Mx
>     Reporter: Kevin Williams
>     Assignee: Brent Daniel
>      Fix For: Java-Mx

>
> It is apparent that we need to go back and refactor after adding the DAS 
> interface (formerly CommandGroup) and the addition of the ConfigHelper.  I 
> think we can base this refactoring on the following assumptions:
>   1. Config can be built up with the ConfigHelper
>   2. Config can be loaded from a Config file
>   3. Config can only be applied to a DAS (via a factory method)
>   4. Commands can only be got from a DAS
> Some specific items we should consider:
>   1. Connection instance should be provided to a DAS instance via a factory 
> method
>   2. SDO ObjectModel is really part of the configuration and should be 
> specified in the config rather than in a factory method or a setter on Command
>   3. Keys are defined in the Config and so "Key" can be removed from the 
> interface package
>   4. Parameter can be removed from the interface package
>   5. SDODataTypes just labels and caches SDO Data Types from SDO TypeHelper 
> and can be removed
>   6. Converter can be removed from Config.ResultDesriptor
>   7. "addParameter" method can be removed from Command
>  

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to