That's a fair point, Pinaki. That is, don't make a lot of these "code relaxations" until we're sure that they are needed. Don't get carried away. Make them as needed. Don't make them just because we think we need them. That way, we can be clearer on the intent of the changes as we commit them. Makes sense.
Kevin On Jan 9, 2008 1:41 PM, Pinaki Poddar <[EMAIL PROTECTED]> wrote: > Hello Daniel, > Relax access restrictions on JDBCStoreManager or whichever parts you > deem necessary to support non-JDBC based access to DB2. My only > suggestion is do it by-parts, isothermally, on-demand, as "More detail > information can be provided once it becomes available" becomes true. > My concern is, otherwise, this patch may set a precedence of > large-scale, anticipatory changes without specific use cases scenarios. > > > > -----Original Message----- > From: Daniel Lee [mailto:[EMAIL PROTECTED] > Sent: Wednesday, January 09, 2008 1:05 PM > To: [email protected] > Subject: Re: [jira] Commented: (OPENJPA-477) making StoreManager more > flexible and extensible. > > Right, IBM has an alternate interface (other than JDBC) to access DB2. > In order to use OpenJPA thru that interface in a project, we need to > sub-class JDBC StoreManager. More detail information can be provided > once it becomes available. Thanks. > > On 1/8/08, Pinaki Poddar (JIRA) <[EMAIL PROTECTED]> wrote: > > > > > > [ > > https://issues.apache.org/jira/browse/OPENJPA-477?page=com.atlassian.j > > ira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=125 > > 56965#action_12556965] > > > > Pinaki Poddar commented on OPENJPA-477: > > --------------------------------------- > > > > "a recent practice for providing a proprietary extension to use an > > interface other than JDBC" - elaboration on this extension will help > > everyone to appreciate the reasons for this patch better -- especially > > > when the changes required by this extension is not only confined at > > StoreManager but permeates to almost every tier of OpenJPA > > (StateManager, Broker, Configuration...). > > > > > > > > > making StoreManager more flexible and extensible. > > > ------------------------------------------------- > > > > > > Key: OPENJPA-477 > > > URL: > https://issues.apache.org/jira/browse/OPENJPA-477 > > > Project: OpenJPA > > > Issue Type: Improvement > > > Components: kernel > > > Affects Versions: 1.0.2 > > > Reporter: Daniel Lee > > > Fix For: 1.0.2, 1.1.0 > > > > > > Attachments: OPENJPA-477.patch > > > > > > > > > The OpenJPA StoreManager needs some modifications to make it more > > flexible and extensible. For example, some private methods and > > attributes need to made either public or protected. The attached > > patch identifies the areas that have been identified thus far. Please > > > let me know if there are any concerns. Many thanks in advance. > > > > -- > > This message is automatically generated by JIRA. > > - > > You can reply to this email to add a comment to the issue online. > > > > > > Notice: This email message, together with any attachments, may contain > information of BEA Systems, Inc., its subsidiaries and affiliated > entities, that may be confidential, proprietary, copyrighted and/or > legally privileged, and is intended solely for the use of the individual or > entity named in this message. If you are not the intended recipient, and > have received this message in error, please immediately return this by email > and then delete it. >
