Skip to content

Commit

Permalink
MONDRIAN: Fix test CompatibilityTest.testNullCollation on Oracle. Native
Browse files Browse the repository at this point in the history
    TopCount was generating SQL that ordered NULL values incorrectly.

[git-p4: depot-paths = "//open/mondrian/": change = 12903]
  • Loading branch information
julianhyde committed Jun 28, 2009
1 parent 967c85a commit 2b67d56
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/main/mondrian/rolap/RolapNativeTopCount.java
Expand Up @@ -70,9 +70,9 @@ public void addConstraint(
String alias = sqlQuery.nextColumnAlias();
alias = dialect.quoteIdentifier(alias);
sqlQuery.addSelect(orderBySql, alias);
sqlQuery.addOrderBy(alias, ascending, true, false);
sqlQuery.addOrderBy(alias, ascending, true, true);
} else {
sqlQuery.addOrderBy(orderBySql, ascending, true, false);
sqlQuery.addOrderBy(orderBySql, ascending, true, true);
}
}
super.addConstraint(sqlQuery, baseCube, aggStar);
Expand Down

0 comments on commit 2b67d56

Please sign in to comment.