In-memory IceStorm

vukicevicvukicevic Member Vladimir VukicevicOrganization: DUGProject: DUG Insight ✭✭
For what it's worth, this patch pulls out the Freeze dependency from IceStorm, and moves it into a set of classes (in StormStorage.h) that implement either a Freeze-backed storage or a STL/in-memory backed storage. The IceStorm-related tests all pass with both the default config and with IceStorm.StorageType=memory .

This makes working with many thousands of topics easier, though this might be moot once the IceStorm freeze map is combined into one map instead of one per topic. Patch is attached.

Comments

  • michimichi Member Michi HenningOrganization: Triodia TechnologiesProject: I have a passing interest in Ice :-) ✭✭✭
    My apologies for the long silence...

    We'll draw up some sort of copyright release agreement, so we can do this properly. (Unfortunately, we have to jump through those legal hoops.)

    So, please bear with us until we get this done.

    Cheers,

    Michi.
  • vukicevicvukicevic Member Vladimir VukicevicOrganization: DUGProject: DUG Insight ✭✭
    No problem :)

    Let me know as well if you'd prefer a different approach to implementing this; this implementation was the least-intrusive I could come up with, though there's probably a better way.
Sign In or Register to comment.