Archived

This forum has been archived. Please start a new discussion on GitHub.

In-memory IceStorm

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

  • 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.
  • 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.