[ http://issues.apache.org/jira/browse/TUSCANY-483?page=all ]
Brent Daniel updated TUSCANY-483:
---------------------------------
Attachment: tuscany438b.txt
Attaching a patch file that contains only the changes to the BigBank sample
required by the API change here.
> 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
> Attachments: tuscany438.txt, tuscany438b.txt
>
> 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]
