22

The question is: how do I make GORM generate left joins instead of inner joins in this particular example?

Testbed:

Given classes A, B and C:

class A{
    B someObject
}

class B{
    C importantObject
}

class C{
    boolean interestingFlag
}

I want to list all the elements of A class that:

  • their B.C object is null OR
  • their B.C object interestingFlag value is false

What I tried so far:

This approach produces correct list of A where B.C is null (conditional 2 commented out) OR correct list of A where B.C.interestingFlag = false (no matter if conditional 1 is commented out or not). When both conditionals are uncommented it returns only a list of elements where A.B.C.interestingFlag = false (A.B.C = null conditional is ignored)

// approach 1 (conditional 1 is ignored)
def result = A.withCriteria{
    someObject{
        or{
            isNull('importantObject') // conditional 1, works well when conditional 2 is commented out
            importantObject{
                eq('interestingFlag', false) // conditional 2, works well alone, discards conditional 1 when both of them are uncommented
            }
        }  
    } 
}

Edit: As requested in comment I'm pasting a hibernate generated sql:

Hibernate: select this_.id as id1_2_, this_.version as version1_2_, 
this_.some_object_id as some3_1_2_, someobject1_.id as id2_0_, 
someobject1_.version as version2_0_, someobject1_.important_object_id as 
important3_2_0_, importanto2_.id as id0_1_, importanto2_.version as version0_1_, 
importanto2_.interesting_flag as interest3_0_1_ from a this_ 
inner join b someobject1_ on this_.some_object_id=someobject1_.id 
inner join c importanto2_ on someobject1_.important_object_id=importanto2_.id 
where ((someobject1_.important_object_id is null or (importanto2_.interesting_flag=?)))

When I copy and paste it in the pgAdmin query tool directly with a few things changed (inner joins changed to left joins, and provided the interestingFlag = "false" parameter) everything works as I wanted (I get both A.B.C = null and A.B.C.importantFlag = false objects)

Hibernate: select this_.id as id1_2_, this_.version as version1_2_, 
this_.some_object_id as some3_1_2_, someobject1_.id as id2_0_, 
someobject1_.version as version2_0_, someobject1_.important_object_id as 
important3_2_0_, importanto2_.id as id0_1_, importanto2_.version as version0_1_, 
importanto2_.interesting_flag as interest3_0_1_ from a this_ 
left join b someobject1_ on this_.some_object_id=someobject1_.id 
left join c importanto2_ on someobject1_.important_object_id=importanto2_.id 
where ((someobject1_.important_object_id is null or (importanto2_.interesting_flag=false)))
2
  • 1
    Try enabling logSql=true in your data source and see what sql this qquery generates. Post it here, please. Commented Jun 13, 2013 at 9:11
  • @TomaszKalkosiński as requested here are the hibernate sql outputs. I know how to tweak the query to work as I want to. I don't know how to make GORM produce the correct query Commented Jun 13, 2013 at 9:57

3 Answers 3

43

Tested and working solution:

    def result = A.withCriteria{
        createAlias('someObject', 'so', CriteriaSpecification.LEFT_JOIN)
        createAlias('so.importantObject', 'imp', CriteriaSpecification.LEFT_JOIN)
        or {
            isNull('so.importantObject')
            eq('imp.interestingFlag', false)
        } 

    }

Solution update as suggested in comment:

    def result = A.withCriteria{
        createAlias('someObject', 'so', JoinType.LEFT_OUTER_JOIN)
        createAlias('so.importantObject', 'imp', JoinType.LEFT_OUTER_JOIN)
        or {
            isNull('so.importantObject')
            eq('imp.interestingFlag', false)
        } 

    }
1
5

Use left join to achieve this. This should work, but I haven't tested it live.

def result = A.withCriteria{
    someObject {
        createAlias("importantObject", "io", CriteriaSpecification.LEFT_JOIN)
        or{
            isNull('importantObject') // conditional 1
            eq('io.interestingFlag', false) // conditional 2            
        }  
    } 
}

Eidt: based on this post: http://grails.1312388.n4.nabble.com/CriteriaBuilder-DSL-Enhancements-td4644831.html this should also work:

def result = A.withCriteria{
    someObject {
        isNull('importantObject') // conditional 1
        importantObject(JoinType.LEFT) {
            eq('interestingFlag', false) // conditional 2
        }  
    } 
}

Please post your results on both solutions.

Edit 2: This is a query that is exactly a situation you've described, but differs from your examples. You must start from here, use showSql to debug generated SQLs and fiddle with left joins.

def result = A.withCriteria{
    or {
        isNull('someObject')
        eq('someObject.importantObject.interestingFlag', false)
    } 
}
7
  • They both generated errors. Solution 1: Class org.hibernate.QueryException Message could not resolve property: importantObject of: A. Solution 2: groovy.lang.MissingMethodException Message No signature of method: AController.importantObject() is applicable for argument types: (javax.persistence.criteria.JoinType, AController$_list_closure1_closure2_closure3) values: [LEFT, AController$_list_closure1_closure2_closure3@40230d] Commented Jun 13, 2013 at 13:08
  • This is strange, since messages indicate errors on A object, while importantObject is in someObject closure. As for solution 1 try to createAlias on someObject and see if that helps. Commented Jun 13, 2013 at 13:23
  • No luck after changing alias to createAlias("someObject.importantObject", "io", CriteriaSpecification.LEFT_JOIN). Now it can't find "io() method" Commented Jun 13, 2013 at 13:41
  • @AndrzejBobak I think there is something wrong in your question and it affects your queries too. Structure you presented doesn't match your question and queries. We can't move on without correct structure so we can fix your queries. Commented Jun 13, 2013 at 13:46
  • This is the EXACT copy of what is in my sample project. Setting it up for the purpose of the question and generating the sql output took me around 10 minutes. I use Grails 2.2.1 and Postgresql 8.3. If you're interested I can send you a copy of this project but it's so tiny it can be created in a matter of a few minutes (3 classes, each with 1 field, generated views + scaffolds). I really don't understand what you mean by "structure I presented doesn't match my question and queries". Commented Jun 13, 2013 at 14:05
1

I was able to achieve left outer joins only with HQL

Class Transaction {
    String someProperty
    static hasMany = [reviews: Review]
    static hasOne = [reviewQueue: ReviewQueue]
}

Class ReviewQueue {
   Transaction transaction
   Boolean isComplete
   Boolean isReady
}

Class Review {
  Transaction transaction
  String reviewResult
}

def list = Transaction.executeQuery(
    "select  t.id, rq.isReady, rq.isComplete, count(r.transaction.id) " +
    "from Transaction t " +
    "join t.reviewQueue rq " +
    "left outer join t.reviews r " +
    "where rq.isComplete = false " +
    "and rq.isReady = true " +
    "group by t.id " +
    "having count(r.transaction.id) = 0 " +
    "order by rq.transaction.id ",
[max: 10, offset: 0])

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.