Hi all, > > Hi, > > integrating that feature sounds great to me. I could definitly use it > in our current project. Why would it not make sense to add it ?
Here is my original argument: I said, of course it may be useful feature. On the other hand it would be only a shortcut for functionality that can be achieved with little pain with the existing PB API. All you have to to is to add the criteria for the foreign keys to the selection criteria and perform a PB.getCollectionByQuery(...). The idea of a microkernel is to provide only methods that cannot substituted by several simpler operations. As explained above, getChildCollectionByCriteria(...) can be replaced very easily. Thus it is a "violation" of the micro-kernel approach. Of course there are several places where OJB is violating the micro-kernel pattern already. (e.g. PB.retrieveReference(...)) SO it will will not be easy for me to convince you that it is important to keep the PB API lean and mean... cheers, Thomas > Michael > > > -----Urspr�ngliche Nachricht----- > Von: Thimo K�nig [mailto:[EMAIL PROTECTED]] > Gesendet: Mittwoch, 16. Oktober 2002 12:42 > An: [EMAIL PROTECTED]; [EMAIL PROTECTED] > Betreff: Child Collections matching an additional Criteria > > > Hi, > > we are using OJB and did some changes to the PersistenceBrokerImpl to > retrieve ChildCollections '#getChildCollectionByCriteria' with an > additional Criteria. That means we are no able to retrieve > only a subset > of an relation. > > The complete integration of this feature needs some more changes e.g > PersistenceBroker, PersistenceBrokerBean, PersistenceBrokerClient, > RequestProcessor. I'm sure I forgot something. > > We've already discussed this with Thomas Mahler. He prefers to don't > integrate it. But he also said that maybe he would change his mind if > there is a serious demand. > > Because we need to integrate it anyway we'd like to here the > opinion of > the Developers and Users. No need to say that the integration would > simplify upgrading to upcoming OJB versions for us. > > Awaiting your comments > Thimo > > -- > To unsubscribe, e-mail: > <mailto:[EMAIL PROTECTED]> > For additional commands, e-mail: > <mailto:[EMAIL PROTECTED]> > > > > -- > To unsubscribe, e-mail: > <mailto:[EMAIL PROTECTED]> > For additional commands, e-mail: > <mailto:[EMAIL PROTECTED]> > > -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
