You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@jackrabbit.apache.org by Dave Brosius <db...@mebigfatguy.com> on 2013/07/25 17:04:00 UTC

Curious code in QueryEngine.execute

QueryEngine.execute(JoinMerger merger,
             ConstraintSplitInfo csInfo, boolean isOuterJoin,
             int printIndentation)

does

if (leftRows == null || leftRows.isEmpty()) {
             return merger.merge(new RowIteratorAdapter(leftRows),
                     new RowIteratorAdapter(new TreeSet<Row>()), null, 
rightCo);
         }


which looks to me like a completely inverted if logic guard.

Right?