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

stapi::refresh_ctable doesn't rewrite the TSV cachefile #50

Open
mutability opened this issue Mar 17, 2016 · 6 comments
Open

stapi::refresh_ctable doesn't rewrite the TSV cachefile #50

mutability opened this issue Mar 17, 2016 · 6 comments
Assignees

Comments

@mutability
Copy link

The TSV cachefile gets written once during ::stapi::open_cached, but then it never appears to get updated, even if you're calling ::stapi::refresh_ctable.

So for long-running processes that periodically refresh, the cache rarely gets used.

@resuna resuna self-assigned this Mar 27, 2016
@resuna
Copy link
Member

resuna commented Apr 5, 2016

With a long-running process, the in-memory speedtable is used. ISTR this was created to avoid having short-running processes beating on the speedtable server.

How would you envision long-running processes using the cache?

@mutability
Copy link
Author

mutability commented Apr 5, 2016 via email

@resuna
Copy link
Member

resuna commented Apr 5, 2016

If the in-core table is completely up to date (re #51 ) then this would be worthwhile doing. If you know the in-core table is valid because you're performing a complete reload or because you know there are no deletions, then updating the table would be worthwhile.

This could be done automatically in reload_ctable or manually (with an update_cache call) after refresh_ctable.

@resuna
Copy link
Member

resuna commented Jul 11, 2016

Should probably have set this to a non-zero value, originally.

  # Default "stale cache" timeout, zero for no timeout.
  variable default_timeout 0

@resuna
Copy link
Member

resuna commented Jul 11, 2016

Have you tried setting the -timeout value (seconds) which is supposed to refresh the tsv file after $timeout seconds when you open it?

@resuna
Copy link
Member

resuna commented Jul 11, 2016

Have a look at bc8513c

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

No branches or pull requests

2 participants