It looks good to me so +1.

My small niggle is the use of

query.getJoins().add(join1);

where I might prefer:

query.addJoin(join1);

which might be easier for users to spot in the API.

If you do stick with getJoins() is there are there good reasons to
make it return a list over a collection? I can't work out if it is
important to force an ordering to joins or not.

Ian
-- 
Ian Turton

------------------------------------------------------------------------------
Simplify data backup and recovery for your virtual environment with vRanger.
Installation's a snap, and flexible recovery options mean your data is safe,
secure and there when you need it. Data protection magic?
Nope - It's vRanger. Get your free trial download today.
http://p.sf.net/sfu/quest-sfdev2dev
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to