From: SourceForge.net <no...@so...> - 2003-02-28 12:32:32
|
Patches item #688610, was opened at 2003-02-18 21:45 You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=428710&aid=688610&group_id=40712 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Konstantin Dmitriev (kdmitriev) Assigned to: Nobody/Anonymous (nobody) Summary: Fetch profiles Initial Comment: Here's my code for fetch profiles. No test case is currently developed (though I have my own). I hope you can test on your own model ---------------------------------------------------------------------- Comment By: Cameron Braid (cameronbraid) Date: 2003-02-28 22:42 Message: Logged In: YES user_id=196175 that is most appreciated. Thanks ---------------------------------------------------------------------- Comment By: Konstantin Dmitriev (kdmitriev) Date: 2003-02-28 18:39 Message: Logged In: YES user_id=693406 SQL scripts to import test data ---------------------------------------------------------------------- Comment By: Cameron Braid (cameronbraid) Date: 2003-02-28 00:18 Message: Logged In: YES user_id=196175 I would like to try this patch. Though to do so I will need to insert a heap of data.. Do you have a dataset that can be added to the patch ? I suggest that you use DBUnit's flat xml data set as they are quite easy to export / import. ---------------------------------------------------------------------- Comment By: Konstantin Dmitriev (kdmitriev) Date: 2003-02-21 18:51 Message: Logged In: YES user_id=693406 Sorry about one-to-one. works now ---------------------------------------------------------------------- Comment By: Konstantin Dmitriev (kdmitriev) Date: 2003-02-20 15:49 Message: Logged In: YES user_id=693406 Hello Chris 1. According to Max R. Andersen's request, fetch profiles are defined in a way similar to sessions.setClassFectchProfiles (class1, "property1, property2"); i.e. not in the query 2. The whole thing is not about transparent eager loading lazy-loading proxies. It's about avoiding N+1. Quite different. I suggest you have a look at the patch and test cases Thank you very much for your feedback Best regards, Konstantin Dmitriev ---------------------------------------------------------------------- Comment By: Chris Nokleberg (herbyderby) Date: 2003-02-20 07:41 Message: Logged In: YES user_id=594737 Instead of parsing queries to determine what fields need to be fetched, I suggest that fetch profiles only be allowed for lazy-loading proxies. Accesses to the underlying getters could be intercepted, mapped to a profile, and loaded if necessary. Basically breaking up a lazy entity into distinct subsets of properties, each of which get loaded independently just-in-time. ---------------------------------------------------------------------- Comment By: Konstantin Dmitriev (kdmitriev) Date: 2003-02-19 23:53 Message: Logged In: YES user_id=693406 - added support for Maps - added support for Collections of values - various mappings added to the test model: list, map, bag, collection of values, one-to-one, many-to-many, component one-to-one present a problem, because lazy load is not supported ---------------------------------------------------------------------- Comment By: Konstantin Dmitriev (kdmitriev) Date: 2003-02-18 22:38 Message: Logged In: YES user_id=693406 Now with the test case. It's for Oracle (Is it a problem?) ---------------------------------------------------------------------- Comment By: Konstantin Dmitriev (kdmitriev) Date: 2003-02-18 22:28 Message: Logged In: YES user_id=693406 Where is the archive? Strange! Attaching again ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=428710&aid=688610&group_id=40712 |