On 06/08/2008, at 11:32 AM, Andrus Adamchik wrote:

On Aug 5, 2008, at 9:24 PM, Lachlan Deck wrote:

Was just thinking that there didn't seem to be any hook available to createSqlString() (which calls the above method) to ensure suppressingDistinct is true for this case.

Sure. I am +1 on such change if we find that overriding it in a subclass for a different adapter is indeed helpful.

Actually it appears to me that perhaps the bit we want to subclass is dbRelationshipAdded(?) which sets forcingDistinct to true. However, the various variables in SelectTranslator have no getters/setters. Are you +1 on providing these with accessors?

with regards,
--

Lachlan Deck



Reply via email to