-
Notifications
You must be signed in to change notification settings - Fork 2.1k
should fix #1701 #1718
should fix #1701 #1718
Conversation
@frozeman, thanks! @luclu, @evertonfraga and @alexvandesande, please review this. |
modules/ethereumNode.js
Outdated
@@ -359,7 +359,7 @@ class EthereumNode extends EventEmitter { | |||
// START MAINNET | |||
else { | |||
args = (nodeType === 'geth') | |||
? ['--fast', '--cache', '1024'] | |||
? ['--fast', '--cache', ((process.arch == 'x64') ? '1024' : '512')] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use '===' ;-)
@lucu can you please check the logs in the console, as well as the terminal so i can figure out whats going on on your side? Also OS etc would be great. |
Sorry @frozeman, here is a comparison of |
I don't think that has to do with these changes in they PR as they probable change nothing for your setup. Could you get a clean clone? seems like something on your system is broken? |
This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread. |
should fix #1701. This decreases the cache for arm and 32 bit architectures.