-
Notifications
You must be signed in to change notification settings - Fork 221
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
Review XForms cache architecture #1718
Comments
Also consider:
|
Tasks:
|
Some explanation of how things should work right now:
|
Ideas for better caching:
|
Also, could we better passivate, or even in some case delete server state? See [customer suggestion[(https://basecamp.com/1721271/projects/3269712/messages/25504605), and my reply:
|
Some thoughts on deleting/passivating state when hitting the same form URL more than once. Scenarios:
In general it's not a great idea to delete state because that would mean that your form in the other tab would stop working. For example open the following URL in two tabs: http://demo.orbeon.com/orbeon/fr/orbeon/contact/new The first tab would stop working completely, which is not desirable. For a given session, and a given form path, whatever it might be, we could automatically passivate (but not completely remove) form sessions with the same path. So in my example above:
Maybe a better approach might be one of working with cache weights: instead of passivating state directly, such hits would just reduce the weight of the data in the cache, scheduling them for earlier passivation. Going further, this might work not only in the case of same paths, but for any forms a user might be opening. This might make sense too: the likelihood of a user working on more than one form at a time is not 0, but it is probably low in practice. So maybe a system of weights could work here too. |
2014-10-27:
NOTE: When session expires, size of Ehcache store on disk doesn't shrink. I don't know if this is preventable. |
2014-10-28:
|
We wish to simplify the code if possible, to make intents clearer and to make it easier to reason about the logic which is likely to remain non-obvious. |
Created #1989 with bug found during investigation. |
Closing the "review" part. Open new bugs for new issues. |
In particular, check and test for:
See also #1572, #1973.
The text was updated successfully, but these errors were encountered: