Pentaho Analysis - Mondrian
  1. Pentaho Analysis - Mondrian
  2. MONDRIAN-756

Error in RolapResult.replaceNonAllMembers leads to NPE

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Severe Severe
    • Resolution: Fixed
    • Affects Version/s: 3.1.6 (3.5.2 GA Suite Release)
    • Component/s: None
    • Labels:
      None
    • Notice:
      When an issue is open, the "Fix Version/s" field conveys a target, not necessarily a commitment. When an issue is closed, the "Fix Version/s" field conveys the version that the issue was fixed in.
    • QA Validation Status:
      Validated by QA

      Description


      Function replaceNonAllMembers sets mList in nonAllMembers, but in the next iteraction does a clear(), leading to a NPE later. See comments for code.

        Activity

        Hide
        Pedro Alves added a comment -
        1. in steelwheels.mondrian.xml, replace all hasAll="true" to hasAll="false" <- did you do this? If not, nonAllMembers won't be populated and you won't see that
        Show
        Pedro Alves added a comment - 1. in steelwheels.mondrian.xml, replace all hasAll="true" to hasAll="false" <- did you do this? If not, nonAllMembers won't be populated and you won't see that
        Hide
        Julian Hyde added a comment -
        Oops, sorry I didn't see your comments til now.

        Yes, I did this. It still didn't reproduce. I've checked in the test case in 13688.

        I'll check in the fix and close this bug when we have a test case.
        Show
        Julian Hyde added a comment - Oops, sorry I didn't see your comments til now. Yes, I did this. It still didn't reproduce. I've checked in the test case in 13688. I'll check in the fix and close this bug when we have a test case.
        Hide
        Julian Hyde added a comment -
        I have checked in a fix, change 13718, will be in mondrian-3.2.1. But I cannot close this bug until I have a test case.
        Show
        Julian Hyde added a comment - I have checked in a fix, change 13718, will be in mondrian-3.2.1. But I cannot close this bug until I have a test case.
        Hide
        Julian Hyde added a comment -
        Test case added in change 14627.
        Show
        Julian Hyde added a comment - Test case added in change 14627.
        Hide
        Li Deng added a comment -
        Followed the repro steps above and report display correctly. No error occur.
        Show
        Li Deng added a comment - Followed the repro steps above and report display correctly. No error occur.

          People

          • Assignee:
            Li Deng
            Reporter:
            Pedro Alves
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: