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

"process out of memory" error #15

Closed
jywarren opened this issue Oct 30, 2016 · 8 comments
Closed

"process out of memory" error #15

jywarren opened this issue Oct 30, 2016 · 8 comments
Milestone

Comments

@jywarren
Copy link

Hi, really interested in using the lib, thanks! Got this error, though, after hanging for a few minutes:

(trusty)warren@localhost:~/sites/plots2$ name-your-contributors ipfs --since=2016-10-15T00:20:24Z
Got response
wrote issue_creators
wrote issue_commenters

<--- Last few GCs --->

  403290 ms: Mark-sweep 701.8 (738.5) -> 699.6 (738.5) MB, 8779.9 / 0 ms [allocation failure] [GC in old space requested].
  413027 ms: Mark-sweep 699.6 (738.5) -> 699.6 (738.5) MB, 9736.6 / 0 ms [allocation failure] [GC in old space requested].
  429222 ms: Mark-sweep 699.6 (738.5) -> 698.8 (738.5) MB, 16195.4 / 13 ms [last resort gc].
  441528 ms: Mark-sweep 698.8 (738.5) -> 697.9 (738.5) MB, 12302.8 / 0 ms [last resort gc].


<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x29d65341 <JS Object>
    1: /* anonymous */(aka /* anonymous */) [/home/warren/.nvm/versions/node/v4.4.0/lib/node_modules/name-your-contributors/node_modules/octokat/dist/node/verb-methods.js:79] [pc=0x25676570] (this=0x29d080dd <undefined>,verbFunc=0x47f8f695 <JS Function module.exports.SimpleVerbs.verbs.create (SharedFunctionInfo 0x3a6b53e5)>,verbName=0x29d25431 <String[6]: create>)
    2: injectVerbMethods [/home/wa...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - process out of memory
Aborted (core dumped)
@jywarren
Copy link
Author

That looks like a lot of data for just the past couple weeks of data; any ideas?

@RichardLitt
Copy link
Member

Hey! Sorry for the massive delay getting back to you on this. Currently on vacation // hacking in Japan, haven't had much time. I think we've fixed this issue elsewhere, but I'm not sure I merged it in. Give me a couple of days to take a look at it.

@RichardLitt RichardLitt modified the milestone: Polish name-your-contributors Nov 18, 2016
@daviddias
Copy link

daviddias commented Jan 3, 2017

Just encountered the same problem FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory. You might want to consider to have some sort of streaming data parsing to having running into situations like these.

@RichardLitt
Copy link
Member

Can you give me your command so I can test it?

@jywarren
Copy link
Author

jywarren commented Jan 8, 2017

I'm using:

var nameYourContributors = require('name-your-contributors');

nameYourContributors('ipfs', {
  since: '2016-12-01T00:00:00Z'
});

with prepended env variables in the node nyc.js. Thanks!

@jywarren
Copy link
Author

jywarren commented Jan 8, 2017

But also the above-referenced direct CLI method:

(trusty)warren@localhost:~/$ name-your-contributors ipfs --since=2016-10-15T00:20:24Z

@RichardLitt
Copy link
Member

Hey! So! I found @tgetgood, and with his help, this issue is no longer an issue! 🎉

This should now be fixed as of 3.0.0, which uses GraphQL to stop all overfetching issues.

@jywarren Try it again? @diasdavid Feel free to use it for IPFS, too. :)

@jywarren
Copy link
Author

jywarren commented Oct 9, 2017 via email

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