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

2GB of memory isn't enough #27

Open
TLyngeJ opened this issue Mar 8, 2017 · 4 comments
Open

2GB of memory isn't enough #27

TLyngeJ opened this issue Mar 8, 2017 · 4 comments

Comments

@TLyngeJ
Copy link

TLyngeJ commented Mar 8, 2017

I tried this plugin on a Drupal project and the generation of the index seems to fail, when PHP tried to allocate more than two GB of memory.

Didn't make much sense to begin with cause I've set the PHP memory limit to -1 in the CLI ini.

So I did a grep -R 'ini_set(' from the ~/.composer/vendor/mkusher folder and realised that the memory is fixed to two GB in padawan/bin/padawan.

Why is that, and would there be a way to increase it, without hacking the PHP component?

@mkusher
Copy link
Member

mkusher commented Mar 8, 2017

hm, I can't say that this limit was set "on purpose", looks like it better to remove it from bin

@mkusher
Copy link
Member

mkusher commented Mar 8, 2017

@TLyngeJ please, have a look padawan-php/padawan.php#97

@TLyngeJ
Copy link
Author

TLyngeJ commented Mar 8, 2017

Yeah, I believe that should cut it.

memory_limit also set in the padawan-server, but I suppose that it's safe enough to assume that it wont go near that limit.

@mkusher
Copy link
Member

mkusher commented Mar 8, 2017

@TLyngeJ well, I've merged fix to master, so it should work well now

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

2 participants