Skip to content

Commit

Permalink
Axis data output may deviate from expected order, as with this test q…
Browse files Browse the repository at this point in the history
…uery:

select {[Cost Type].[current], [Cost Type].[planned]} * 
 {Tail([Time].[Year].Members,3)} ON COLUMNS,
          NON EMPTY Crossjoin({[Planner].CurrentMember.Children},
 {[Part].[All Parts].[M].Children}) ON ROWS
     from Inventory
       where ([Measures].[Total Cost]);

where the years are ordered as 2005, 2006, 2004 instead of 2004, 2005, 2006. 

The RolapResult.mergeAxes method starts iterating through the pl1 list when arrayLen == -1, then continues to use this same list in subsequent code to build a new member list for the axis.  This change resets to the start of the pl1 list so that the new member list is built from the beginning of the pl1 list.

[git-p4: depot-paths = "//open/mondrian/": change = 9622]
  • Loading branch information
Chuck Reidmiller committed Jul 19, 2007
1 parent 9469bbc commit 4b3de92
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/main/mondrian/rolap/RolapResult.java
Expand Up @@ -1777,6 +1777,8 @@ static Axis mergeAxes(Axis axis1, Axis axis2) {
}
break;
}
// reset to start of List
// pl1 = axis1.getPositions();
}
if (arrayLen == 1) {
// single Member per position
Expand Down

0 comments on commit 4b3de92

Please sign in to comment.