Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Readonly cache #57

Closed
wants to merge 4 commits into from
Closed

Readonly cache #57

wants to merge 4 commits into from

Conversation

jpark800
Copy link
Contributor

@jpark800 jpark800 commented Jun 3, 2019

First step in the cache support. Fixes #53.

@balamg
Copy link
Collaborator

balamg commented Jun 7, 2019

What you need is an on-event hook..(called "pre-processor" in BE) This is the place where you can load necessary tuples from the store. Also this needs to be integrated with the rule session api with a "Store" interface.
Incidentally I have a WIP branch that addresses the store interface and some more (like an interface to externalise join tables themselves and an in-memory implementation of those) Perhaps you can use that as the starting point for implementing your Solr/Redis store?

@balamg
Copy link
Collaborator

balamg commented Jun 11, 2019

Take a look at the 'store' branch to start with.

@jpark800
Copy link
Contributor Author

@balamg Sure, will take a look.

@jpark800 jpark800 removed the request for review from rameshpolishetti June 12, 2019 05:10
@jpark800
Copy link
Contributor Author

jpark800 commented Jun 22, 2019

After reviewing the WIP store branch, it would be best to merge the two branches first. The merge would include a part of:

  • redis tuple store with update support
  • TupleType based secondary index in redis
  • multi store support(per tupletype store)
  • pre-processor(or startup) time tuple loading
  • tuple update propagation to clustered engines.

Thus closing the PR.

@jpark800 jpark800 closed this Jun 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

read-only cache support
2 participants