StackoverFlow error when querying for multiple records using OR clause

[sureshpendap]

I am invoking this method with 2000 keys, basically I would like to retrieve 2000 records matching these alternate keys using a AK_COL = :key1 OR AK_COL=key2.....
etc
AlternateKeyValue is an abstraction which encapsulates the fieldNames comprising my Alternate key and their values.

public Collection<QueryBase> query(Collection<AlternateKeyValue> akKeys)
{
    System.out.println("Constructing Query: " + akKeys.size());
    BooleanExpression[] allFieldsMatch = new BooleanExpression[akKeys.size()];

    int aKeyIndex = 0;
    for (AlternateKeyValue aKey : akKeys)
    {
        Set<String> fields = aKey.getFieldNames();
        BooleanExpression[] expressions = new BooleanExpression[fields.size()];

        //Alternate Key can be composite, loop through all the fields
        //and create an equality expression predicate for each field
        int fieldIndex = 0;
        for (String fieldName : fields)
        {

            Object fieldValue = aKey.getFieldValue(fieldName);

            Path<Object> fieldExpr = Expressions.path(Object.class, queryClass, fieldName);
            expressions[fieldIndex++] = Expressions.predicate(Ops.EQ, fieldExpr, Expressions.constant(fieldValue));
        }
        System.out.println("Constructing Expression for : " + aKeyIndex);
        //We need an Entity for which all the fields of a Alternate key match
        allFieldsMatch[aKeyIndex++] = BooleanExpression.allOf(expressions);
    }

    System.out.println("End of LOOP");

    //We need to get all the Entries in bulk using the OR clause
    Predicate anyOfExpr = BooleanExpression.anyOf(allFieldsMatch);

    System.out.println("/-----------***********AFTER  ANYOF..........");
    System.out.println("/-----------***********CHANGES_TAKING EFFECT-----------****************/////");
    QueryBase bulkGetQuery = new JPAQuery(null , JPQLTemplates.DEFAULT)
                                .from(this.queryClass)
                                .where (anyOfExpr);  // For very large values like 5000 this constructor itself throws a Stackoverflow error
    bulkGetQuery.list(this.queryClass);  //This throws a StackOverFlow exception
}   

The Stackoverflow error exception

java.lang.StackOverflowError
at java.lang.Integer.rotateLeft(Integer.java:1128)
at com.google.common.collect.Hashing.smear(Hashing.java:47)
at com.google.common.collect.RegularImmutableSet.contains(RegularImmutableSet.java:49)
at com.mysema.query.jpa.JPQLSerializer.visitOperation(JPQLSerializer.java:393)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:246)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:42)
at com.mysema.query.types.OperationImpl.accept(OperationImpl.java:91)
at com.mysema.query.support.SerializerBase.handle(SerializerBase.java:102)
at com.mysema.query.support.SerializerBase.visitOperation(SerializerBase.java:277)
at com.mysema.query.jpa.JPQLSerializer.visitOperation(JPQLSerializer.java:397)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:246)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:42)
at com.mysema.query.types.OperationImpl.accept(OperationImpl.java:91)
at com.mysema.query.support.SerializerBase.handle(SerializerBase.java:102)
at com.mysema.query.support.SerializerBase.visitOperation(SerializerBase.java:277)
at com.mysema.query.jpa.JPQLSerializer.visitOperation(JPQLSerializer.java:397)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:246)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:42)
at com.mysema.query.types.OperationImpl.accept(OperationImpl.java:91)
at com.mysema.query.support.SerializerBase.handle(SerializerBase.java:102)
at com.mysema.query.support.SerializerBase.visitOperation(SerializerBase.java:277)
at com.mysema.query.jpa.JPQLSerializer.visitOperation(JPQLSerializer.java:397)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:246)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:42)
at com.mysema.query.types.OperationImpl.accept(OperationImpl.java:91)
at com.mysema.query.support.SerializerBase.handle(SerializerBase.java:102)
at com.mysema.query.support.SerializerBase.visitOperation(SerializerBase.java:277)
at com.mysema.query.jpa.JPQLSerializer.visitOperation(JPQLSerializer.java:397)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:246)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:42)
at com.mysema.query.types.OperationImpl.accept(OperationImpl.java:91)
at com.mysema.query.support.SerializerBase.handle(SerializerBase.java:102)
at com.mysema.query.support.SerializerBase.visitOperation(SerializerBase.java:277)
at com.mysema.query.jpa.JPQLSerializer.visitOperation(JPQLSerializer.java:397)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:246)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:42)
at com.mysema.query.types.OperationImpl.accept(OperationImpl.java:91)
at com.mysema.query.support.SerializerBase.handle(SerializerBase.java:102)
at com.mysema.query.support.SerializerBase.visitOperation(SerializerBase.java:277)
at com.mysema.query.jpa.JPQLSerializer.visitOperation(JPQLSerializer.java:397)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:246)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:42)
at com.mysema.query.types.OperationImpl.accept(OperationImpl.java:91)
at com.mysema.query.support.SerializerBase.handle(SerializerBase.java:102)
at com.mysema.query.support.SerializerBase.visitOperation(SerializerBase.java:277)
at com.mysema.query.jpa.JPQLSerializer.visitOperation(JPQLSerializer.java:397)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:246)
at com.mysema.query.support.SerializerBase.visit(SerializerBase.java:42)
at com.mysema.query.types.OperationImpl.accept(OperationImpl.java:91)

[timowest]

You should try something like this instead

BooleanBuilder builder = new BooleanBuilder();
for (String fieldName : fields) {
    Path<Object> fieldExpr = Expressions.path(Object.class, queryClass, fieldName);
    List<Object> fieldValues = new ArrayList<>();
    for (AlternateKeyValue aKey : akKeys) {
        fieldValues.add(aKey.getFieldValue(fieldName);
    }
    builder.or(fieldExpr.in(fieldValues));
}

Could you verify if this works?

[sureshpendap]

I cannot use the IN clause as my Alternate key can be composite composing of multiple columns.
The IN clause with multiple columns is not supported by all the databases.
I would like to construct a query assuming that my AK is (col1, col2) as
select * from my_table where (col1 = :col11 and col2 = :col21) OR (col1 = :col12 and col2 = :col22) ..........................

This should retrieve all the records from my_table which satisfy my collections of Alternate key values.

[timowest]

As a workaround you could create a template expression for this case which looks like
"{0} or {1} or ...", but I will try come up with a better fix.

[sureshpendap]

Can you please elaborate on the above workaround?

[timowest]

Something like this should work

StringBuilder builder = new StringBuilder("{0}");
for (int i = 1; i < args.length; i++) {
    builder.append(" or {" + i + "}");
}
BooleanExpression where = Expressions.booleanTemplate(builder.toString(), args);

args would be here of type BooleanExpression[]. Instead of creating a chain of binary or expressions you can create a single flat expression as a template expression.

Unfortunately this can't be generalized in Querydsl at the core level, since not all Querydsl modules support template expressions (e.g. Lucene and Mongodb).

[timowest]

@sureshpendap Did you try the suggestion?

[sureshpendap]

@timowest No I did not try it. Like I mentioned I have multiple columns composing my key.
So the above would not work as it is, as I need to construct where clause as

(col1 = :col1 and col2=:col2) or (col1=:col3 and col2=:col4) or.......

The expression in between two OR's () or () is actually a conjunction of multiple columns

[timowest]

I think it should work, add additional parenthesis if you need. I am closing this for now, since I don't see a possibility for a fix of this issue on the Querydsl side.