Re: DAS Container - First part ready for review....

2006-11-22 Thread Amita Vadhavkar
Hi Luciano, I followed JIRA-899 and the J2SE (my old client etc.) works fine. But for the war, (even the one from your sandbox, without any changes), I am getting the below error. This error was not there on 20th Nov :), So not sure if this is a result of some other JIRA/compatibility issues? I

Re: DAS Container - First part ready for review....

2006-11-21 Thread Amita Vadhavkar
Hi Luciano, I have made the web app and the das container combination working. There were some class cast etc. problems in orginal code. Now the jsp is showcasing a working SELECT through getAllCompanies(). This work is attached to JIRA-904. Please check it. I can modify it to showcase all CRUD +

Re: DAS Container - First part ready for review....

2006-11-21 Thread Luciano Resende
Just in case, by working with patches I mean to do something like : svn diff /path/to/patchfile.patch This would create a patchfile.patch with only the difference from your code relative to the one on the trunk (on this case sandbox) and I can easily just apply the patch in my end. Let me

Re: DAS Container - First part ready for review....

2006-11-18 Thread Luciano Resende
I have commited some small changes and a simple web app to my sandbox. https://svn.apache.org/repos/asf/incubator/tuscany/sandbox/lresende/ -- Luciano Resende http://people.apache.org/~lresende On 11/17/06, Luciano Resende [EMAIL PROTECTED] wrote: Jim and Kevin, thanks... I'm working on

Re: DAS Container - First part ready for review....

2006-11-17 Thread Kevin Williams
Jim Marino wrote: If I understood your question correctly, DAS allows a 1:1 and 1:N relationship, An application could have multiple components using the same config file, or could split the config file based on the components model... but the choice is really made by the app developer.

Re: DAS Container - First part ready for review....

2006-11-17 Thread Luciano Resende
Jim and Kevin, thanks... I'm working on that... On 11/17/06, Kevin Williams [EMAIL PROTECTED] wrote: Jim Marino wrote: If I understood your question correctly, DAS allows a 1:1 and 1:N relationship, An application could have multiple components using the same config file, or could split

Re: DAS Container - First part ready for review....

2006-11-16 Thread Jim Marino
If I understood your question correctly, DAS allows a 1:1 and 1:N relationship, An application could have multiple components using the same config file, or could split the config file based on the components model... but the choice is really made by the app developer. This sounds similar

Re: DAS Container - First part ready for review....

2006-11-15 Thread Amita Vadhavkar
Hi Luciano, I had done a few changes and will be attaching the working code to JIRA in next couple of hours. Just wanted to update you for same and will start looking at the test case development after that. (Please note, for the basic testing to check that the code is working, I have still kept

Re: DAS Container - First part ready for review....

2006-11-15 Thread Luciano Resende
Hi Amita Sorry for taking little longer, but here are some comments on your updates: I have incorporated your changes to DataAccessComponentTypeLoader Changes to DataAccessEngine.java are assuming that the container should be responsible to setup database, and I don't believe that is the

Re: DAS Container - First part ready for review....

2006-11-14 Thread Venkata Krishnan
Hi, I have tried my best to explain... just in case something is not right in this could somebody pitch in please. Thanks. - Venkat On 11/14/06, Luciano Resende [EMAIL PROTECTED] wrote: Hi All I have updated (Tuscany-904) with a new version of the DAS container. This is based on the

Re: DAS Container - First part ready for review....

2006-11-14 Thread Jim Marino
On Nov 14, 2006, at 8:15 AM, Venkata Krishnan wrote: Hi, I have tried my best to explain... just in case something is not right in this could somebody pitch in please. Thanks. - Venkat On 11/14/06, Luciano Resende [EMAIL PROTECTED] wrote: Hi All I have updated (Tuscany-904) with a

Re: DAS Container - First part ready for review....

2006-11-14 Thread Jeremy Boynes
On 11/14/06, Jim Marino [EMAIL PROTECTED] wrote: One minor thing here. Unless the configuration values are always immutable, an ObjectFactory should be injected otherwise one component could accidentally modify a configuration parameter of another as instances will be shared. When the component

Re: DAS Container - First part ready for review....

2006-11-14 Thread Jim Marino
On Nov 14, 2006, at 12:02 PM, Jeremy Boynes wrote: On 11/14/06, Jim Marino [EMAIL PROTECTED] wrote: One minor thing here. Unless the configuration values are always immutable, an ObjectFactory should be injected otherwise one component could accidentally modify a configuration parameter of

Re: DAS Container - First part ready for review....

2006-11-14 Thread Luciano Resende
I think this begs the question of what DAS configuration is. I'm not sure all configuration is part of a component type since the latter always has a 1:1 relationship with its implementation. For example, I believe the DAS configuration specifies mappings to database tables. Are these mappings

DAS Container - First part ready for review....

2006-11-13 Thread Luciano Resende
Hi All I have updated (Tuscany-904) with a new version of the DAS container. This is based on the initial contribution that Amita did, with some clean-up, refactoring and additions to try to fix a possible problem with mutable properties that are shared among instance (pointed by Jim