Pentaho Analysis - Mondrian

Mondrian does not call SegmentCache.getSegmentHeaders()

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Critical Critical
  • Resolution: Unresolved
  • Affects Version/s: 3.5.0 GA (4.8.0 GA Suite Release)
  • Fix Version/s: 4.1.0 (TBD GA Suite Release)
  • Component/s: Caching
  • Labels:
    None
  • QA Validation Status:
    Not Yet Validated

Description

Using an external SegmentCache via SegmentCacheInjector.addCache(SegmentCache), Mondrian doesn't recognize cache content that was inserted before a server restart.

Looking at source in git master branch, SegmentCache.getSegmentHeaders() is only called within SegmentCacheWorker.getSegmentHeaders(), which doesn't seem to be called anywhere.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Julian Hyde added a comment -
I can see that SegmentCache.getSegmentHeaders() is only called from SegmentCacheWorker.getSegmentHeaders(), but no code calls that. These should be some event where mondrian queries the content of the cache. Maybe that event is where mondrian joins a cache; also, perhaps it should re-sync with the cache if it has received a lot of errors and/or a particular amount of time has elapsed.
Show
Julian Hyde added a comment - I can see that SegmentCache.getSegmentHeaders() is only called from SegmentCacheWorker.getSegmentHeaders(), but no code calls that. These should be some event where mondrian queries the content of the cache. Maybe that event is where mondrian joins a cache; also, perhaps it should re-sync with the cache if it has received a lot of errors and/or a particular amount of time has elapsed.
Hide
Permalink
Will Gorman added a comment -
Please work with QA on creating a regression test to our SpiraTest environment for this issue.
Show
Will Gorman added a comment - Please work with QA on creating a regression test to our SpiraTest environment for this issue.
Hide
Permalink
Pedro Alves added a comment -
Any news here?
Show
Pedro Alves added a comment - Any news here?
Hide
Permalink
Luc Boudreau added a comment -
Pedro, could you send us the code you have used to fix this issue? I remember you mentioned that you found a way to fix this.
Show
Luc Boudreau added a comment - Pedro, could you send us the code you have used to fix this issue? I remember you mentioned that you found a way to fix this.
Hide
Permalink
Pedro Vale added a comment -
hey guys. There's a pending pull request with the fix for this. It's here:

https://github.com/pentaho/mondrian/pull/13

Although this includes a bunch of other stuff, the commit you're looking for is this one:

https://github.com/webdetails/mondrian/commit/9ab54555427f5482c56003cfd3ddff267b8086cc
Show
Pedro Vale added a comment - hey guys. There's a pending pull request with the fix for this. It's here: https://github.com/pentaho/mondrian/pull/13 Although this includes a bunch of other stuff, the commit you're looking for is this one: https://github.com/webdetails/mondrian/commit/9ab54555427f5482c56003cfd3ddff267b8086cc

People

  • Assignee:
    Unassigned User
    Reporter:
    Tiago Gomes Ferreira
Vote (4)
Watch (5)

Dates

  • Created:
    Updated: