On 11/13/06, Allen Gilliland <[EMAIL PROTECTED]> wrote:
One of the things that I am planning to do for the 3.2 release is do some audit/cleanup of the current business layer code. There are a variety of things which could use improving, but the main goal is to fix our Hibernate configuration so that we are 1) properly using the open session in view pattern and 2) enabling lazy fetching on all objects and associations.
All good.
Right now our Hibernate config is pretty messy and doesn't take advantage of many of Hibernate's performance features, so the main reason to do this work is to improve the performance of the business layer. The second big reason is just to reduce clutter and simplify the code as much as possible. There are plenty of places in the code where we have methods that aren't used at all or methods which are duplicated, so those would all be cleaned up. I have most of this work done already (but not checked in) and there aren't really any surprise changes that I had to make except when it came to the hierarchical objects. I tried for multiple days to get the hierarchical objects to work with the updated hibernate config and the current data model, but I kept running into problems. So to fix the problem I had to make a small tweak to the way hierarchical objects are persisted which fixed my issues and I believe drastically simplifies the problem overall. The basic change is that I have completely removed the HierarchicalPersistentObject class and Assoc and it's subclasses and changed the data model so that we have a more normal hierarchical model. So, for weblog categories I added a simple 'parentid' column to the weblogcategory table and that allows a category to manage relationships between it's parent and children directly. Same goes for the FolderData class, but as it turns out that column already existed in the schema but wasn't being used. Upgrade path for both of these is fairly simple and only requires populating these columns with the right value.
The hierarchical persistent object and the assoc tables were there for one purpose: to make it possible to do a recursive query (i.e. query for all entries under /pets and /pets/dogs, /pets/cats etc.) with only one SQL statement. But that hasn't worked since we made the switch to the Hibernate Criteria API back in 2004 and we've been doing fine without it. And the code that started as complex has been moved around so much that now it's almost unintelligible. So I'm +1 with that change. - Dave
