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

Add more disk space usage checks #2206

Closed
sipa opened this issue Jan 23, 2013 · 3 comments
Milestone

Comments

@sipa
Copy link
Member

@sipa sipa commented Jan 23, 2013

Currently, there are only disk space usage checks before writing block data. The largest blobs of data are written when flushing the coindb, though, so this would also need a check.

@gmaxwell

This comment has been minimized.

Copy link
Contributor

@gmaxwell gmaxwell commented Jan 23, 2013

Due to fa45c26 it doesn't even test when writing blocks anymore (because pre-allocation is infrequent it effectively never tests at runtime)

@sipa

This comment has been minimized.

Copy link
Member Author

@sipa sipa commented Jan 23, 2013

@gmaxwell Right, it's only at pre-allocation time (which makes sense, when just looking at block storage), but we still need to prevent out-of-disk-space when doing coincache flushes, as these are large bulk writes.

@gavinandresen

This comment has been minimized.

Copy link
Contributor

@gavinandresen gavinandresen commented Jan 30, 2013

Fixed by pull #2224

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.