Skip to content

add constant for shared (GUI/core) setting -dbcache#3687

Merged
laanwj merged 2 commits intobitcoin:masterfrom
Diapolo:dbcache
Feb 18, 2014
Merged

add constant for shared (GUI/core) setting -dbcache#3687
laanwj merged 2 commits intobitcoin:masterfrom
Diapolo:dbcache

Conversation

@Diapolo
Copy link
Copy Markdown

@Diapolo Diapolo commented Feb 16, 2014

  • adds nDefaultDbCache, nMaxDbCache and nMinDbCache in txdb.h

@sipa
Copy link
Copy Markdown
Member

sipa commented Feb 16, 2014

ACK, if pulltester is happy.

@wtogami
Copy link
Copy Markdown
Contributor

wtogami commented Feb 17, 2014

ACK

1 similar comment
@laanwj
Copy link
Copy Markdown
Member

laanwj commented Feb 17, 2014

ACK

Comment thread src/init.cpp Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could make this nMinDbCache and use it in the GUI as well. AFAIK there is currently no lower bound there.

@Diapolo
Copy link
Copy Markdown
Author

Diapolo commented Feb 17, 2014

I added nMinDbCache and also use it for the GUI now.

@Diapolo
Copy link
Copy Markdown
Author

Diapolo commented Feb 17, 2014

@laanwj Have you got a nice and clean idea, how we could share strprintf(_("Set database cache size in megabytes (%d to %d, default: %d)"), nMinDbCache, nMaxDbCache, nDefaultDbCache) with the Qt settings tooltip (which is currently wrong, as I didn't yet update it)?

@laanwj
Copy link
Copy Markdown
Member

laanwj commented Feb 17, 2014

@Diapolo IMO the default, min and max do not need to be in the tooltip at all, it should just explain what the setting does. We don't need that in the GUI: the limits are enforced by the widget, and the default is already shown by default :)

- adds nDefaultDbCache, nMaxDbCache and nMinDbCache in txdb.h
@Diapolo
Copy link
Copy Markdown
Author

Diapolo commented Feb 17, 2014

I removed the tooltip and unneeded defaults from the .ui file but left the min and max in the description, for par we also have such extended infos for example :).

@BitcoinPullTester
Copy link
Copy Markdown

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/82e96006ae03a7debebb9ad36f939ddabb0d2f9f for binaries and test log.
This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/
Contact BlueMatt on freenode if something looks broken.

@jgarzik
Copy link
Copy Markdown
Contributor

jgarzik commented Feb 17, 2014

ACK

laanwj added a commit that referenced this pull request Feb 18, 2014
82e9600 add constants for shared (GUI/core) -dbcache settings (Philip Kaufmann)
879b390 Increase default dbcache to 100 MiB (Pieter Wuille)
@laanwj laanwj merged commit 82e9600 into bitcoin:master Feb 18, 2014
@Diapolo Diapolo deleted the dbcache branch February 18, 2014 14:04
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants