-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Exception at startup in Celluloid actor on aws-sdk (2.0.47) #833
Comments
Here's another we just noticed, exact same situation.
|
Do you have the ability to run this in Ruby 2.0 or newer? While it certainly is possible autoload is problematic, my understanding is that was resolve, but maybe not in Ruby 1.9.3. |
@trevorrowe This code base has severe memory growth issues on 2.0 or later. |
@trevorrowe Thanks. Does that exist in 2.2.2 as well? Because we have the same issue in both, and I'm pretty confident the problem is in our code, and we leak with the AWS SDK v1 as well... Here's another spurious issue that we've only seen once. Including it here to be helpful, not pedantic. :-)
|
Yes, the memory leak exists in the latest version of Ruby. All of these initialized constant errors are going to be caused by the same issue with autoloading. I'll see about porting the eager autoload utility. |
@trevorrowe Thanks |
I've pushed a commit to the version-2.1-branch of the SDK that adds I'm hoping to release 2.1 later this week or early next week. |
This is now part of the 2.1 public release, available now. |
This seems to be working better, but not yet perfect. Using 2.1.0, and calling:
we're still seeing (at least) this one exception:
I say "at least" because this behavior has always been non-deterministic. |
@trevorrowe After a few hours of testing, I can confirm that the exception above is very repeatable in my case, and is the only such issue we've seen in many iterations. |
The fix for eager_autoload! exposed an issue with autoloading optional dependencies eagerly. Travis caught these errors when testing against Ruby installations that did not have these optional dependencies present. See #833
@tmornini The |
@trevorrowe Thanks, looking forward to next release. |
Hey there.
Reporting some odd behavior we're seeing during startup of a Celluloid based app.
This issue is mostly non-deterministic:
We've been upgrading for v1 -> v2 and this reminds me of AWS SDK v1 issues solved with #eager_autoload! and this same code triggered a common exception experienced in threads in ActiveSupport (undefined method `encode' for ActiveSupport::JSON:Module)
The text was updated successfully, but these errors were encountered: