-
Notifications
You must be signed in to change notification settings - Fork 9
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
max_old_space_size too small and not configurable? #34
Comments
Wow, that's crazy. I only end up using about 1.5GB. Can you give me the operating system, 64bit/32bit, node version and filename? |
update: ya increasing the value that is hardcoded in |
It's 64-bit on node v8.4 on a machine with 128GB RAM:
Here's a series of snapshots from
(That run succeeded) |
Which file are DTD/TTIS file are you using to generate the GTFS file? |
It appears to be RJTTF921.ZIP |
I've run it through locally and not had any issues. Is there any chance you can update your node version? |
@i've been getting the
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
memory error and traced it to the node max_old_space_size argument.This appears to be set to 3000 in ./bin/dtd2mysql.sh
I tried increasing to 4096 and then finally to 16384 and that seems to have solved it.
This was from calling dtd2mysql as follows:
$ DATABASE_USERNAME=dtd2mysql DATABASE_NAME=timetable NODE_OPTIONS='--max_old_space_size=16384' /usr/bin/ts-node ./src/index.ts --gtfs-zip ~/gtfs.zip
However, when I run it on the system-installed version, as follows, it seems to still get the error:
$ DATABASE_USERNAME=dtd2mysql DATABASE_NAME=timetable NODE_OPTIONS='--max_old_space_size=16384' dtd2mysql --gtfs-zip ~/gtfs.zip
Which I presume is due to the presence of
--max_old_space_size=3000
in/usr/bin/dtd2mysql
(just checking that now...)The text was updated successfully, but these errors were encountered: