Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

app should shutdown nicely #59

Closed
fix opened this issue Apr 20, 2016 · 7 comments
Closed

app should shutdown nicely #59

fix opened this issue Apr 20, 2016 · 7 comments
Assignees

Comments

@fix
Copy link
Contributor

fix commented Apr 20, 2016

Otherwise, the blockchain.db is regularly messed up and need to be rebuilt from ground up.

Also the blockchain verification is far from optimised, with exponantial time computation...
So far the "verify" flag is forced in the code, even configured otherwise in config.json. Should be reverted asap in my opinion.

@sigwo
Copy link

sigwo commented Apr 21, 2016

Maybe wrap in Node.js native event system?

process.on( 'SIGINT', function() {
  console.log( "\nGracefully shutting down from SIGINT (Ctrl-C)" );
  // some other closing procedures go here
  process.exit( );
})

@Phoenix1969
Copy link

Phoenix1969 commented Apr 23, 2016

Yes, this happened to me just now... so upsetting after waiting hours for sync... all i did was enter the passphrase and restart, now back to scratch!

@LiskDevs
Copy link

Happened to me several times too

@Odsejen
Copy link

Odsejen commented Apr 23, 2016

I found another strange issue with the blockchain.db, tried to make a backup from the *.db file, but even when i'm trying to make a backup (lisk is stopped before), it won't be in sync when i start it again, what means that the node will start from block Zero, the same issue of course when i'm trying to change the original blockchain.db with the backup.db file. I tried to reboot, before i'm doing the backup copy, but it makes no difference (all tested without lisk.sh autostart function).
I think it's a strange issue, or i'm overseeing something?
Will continue to find more...

@mrv777
Copy link
Contributor

mrv777 commented May 10, 2016

Does this still apply with postgresql?

@karmacoma karmacoma self-assigned this May 10, 2016
@Isabello
Copy link
Contributor

This should no longer be an issue since it was related to SQLite.

@fix
Copy link
Contributor Author

fix commented May 17, 2016

I think we can close it for now with the new version. Should it happen again, please reopen it

@fix fix closed this as completed May 17, 2016
MaciejBaj added a commit that referenced this issue Apr 10, 2018
Add exceptions for mainnet - Related #57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

8 participants