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

rpc save command causes wallet timeout #50

Closed
ghost opened this issue Jun 21, 2014 · 3 comments
Closed

rpc save command causes wallet timeout #50

ghost opened this issue Jun 21, 2014 · 3 comments

Comments

@ghost
Copy link

ghost commented Jun 21, 2014

Currently the RPC save blockchain command communicates to the daemon and waits for a response; however, because blockchain storage at the daemon takes a long time, the wallet always timesout waiting for the RPC to respond and freezes the wallet.

Proper wallet behaviour for save blockchain functionality should prevent the wallet from timing out while it waits for the daemon to complete saving the blockchain.

@Neozaru
Copy link
Contributor

Neozaru commented Jun 22, 2014

The daemon should indeed respond as soon as the request is accepted, a not after the blockchain storage is completed. This method was implemented when the blockchain storage took less than 2 secs.
I don't know if the best thing to do is to change the behavior of this RPC call, or to disable it completely from the Wallet : This is not the role of the Wallet to send directives to the daemon, and the blockchain storage will be migrated to a more efficient system (database)

@Jojatekok
Copy link
Contributor

Storing the blockchain should not overwrite the whole file, but should append to it (like in a DB, as @Neozaru mentioned above). Thus, even automatic async savestates could be allowed each 'n' blocks/minutes (with a new command-line argument --autosave [arg]).

@fluffypony
Copy link
Contributor

Fixed in rpcwallet, although a more thorough fix will be with the move to BlockchainDB

stoffu pushed a commit to stoffu/monero that referenced this issue Aug 10, 2018
…plate

core: cache block template where possible /monero#4146
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

3 participants