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

Memory allocation problems since 66f612cb80028de2f03f80fa13a84fa8d89c2b42 #74

Open
creatoro opened this issue Apr 28, 2010 · 2 comments

Comments

@creatoro
Copy link

I get memory exhausted problems when performing longer queries since SHA: 66f612c (attach the Sprig object to the Field object for cross referencing).

The same query that caused this memory allocation problem with this Sprig version didn't have problems with earlier Sprig versions.

@sfgeorge
Copy link
Contributor

sfgeorge commented Aug 5, 2010

I completely agree. Excellent catch! At first, I thought this was restricted to affect the php 5.2.x crows, because of a 5.2.x cyclical reference bug (http://bugs.php.net/bug.php?id=48690). However, I was able to reproduce the same problem even in 5.3.1.

Since the cross reference ($object property) has not been officially used yet, the simplest solution for the time being is to remove it.

@sfgeorge
Copy link
Contributor

sfgeorge commented Aug 5, 2010

This is now fixed in 6c560a1.

Wanted to uncouple Sprigs and their fields from each other in their destructors by manually unsetting all properties, but it didn't do the trick.

Instead, as mentioned above, the short term solution is to remove the cross reference ($object) variable.

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