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

Temporary data is not pruned #1004

Closed
ddfreyne opened this Issue Nov 30, 2016 · 4 comments

Comments

Projects
None yet
2 participants
@ddfreyne
Member

ddfreyne commented Nov 30, 2016

Nanoc does not remove unused entries from data files in the tmp/ directory.

These files tend to grow over time and will consume more and more memory, as they’re always fully loaded into memory.

@ddfreyne

This comment has been minimized.

Show comment
Hide comment
@ddfreyne

ddfreyne Dec 8, 2016

Member

A solution that seems obvious to me at the moment is to not load data (checksums, compiled content, and dependencies) for items and layouts that don’t exist anymore.

Member

ddfreyne commented Dec 8, 2016

A solution that seems obvious to me at the moment is to not load data (checksums, compiled content, and dependencies) for items and layouts that don’t exist anymore.

ddfreyne added a commit that referenced this issue Dec 17, 2016

Merge pull request #1013 from nanoc/gh-1004-compiled-content-cache
Prune old data from compiled content cache
@ddfreyne

This comment has been minimized.

Show comment
Hide comment
@ddfreyne

ddfreyne Dec 17, 2016

Member

#1013 fixes the issue for the compiled content cache. Checksums still need to be pruned. Dependencies are already taken care of.

Member

ddfreyne commented Dec 17, 2016

#1013 fixes the issue for the compiled content cache. Checksums still need to be pruned. Dependencies are already taken care of.

@connorshea

This comment has been minimized.

Show comment
Hide comment
@connorshea

connorshea commented Dec 18, 2016

👏 thanks @ddfreyne :D

@ddfreyne

This comment has been minimized.

Show comment
Hide comment
@ddfreyne

ddfreyne Dec 21, 2016

Member

Checksums are pruned as of #1027, so this issue is resolved.

Member

ddfreyne commented Dec 21, 2016

Checksums are pruned as of #1027, so this issue is resolved.

@ddfreyne ddfreyne closed this Dec 21, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment