-
Notifications
You must be signed in to change notification settings - Fork 755
Timing out after 1.0 -> 1.1 upgrade #334
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
Comments
Hmm, thanks for reporting this. Are you using server rendering? If so, maybe it was caused by our changes there. We switched the default pool size from 10 to 1 but that shouldn't have affected total time, because the 10-member "pool" was actually running one-at-time anyways (unless you're on JRuby). I've seen a couple bugs in the past where sprockets is re-compiling things in production (A server-rendering bug I think) it could be that? If you're not using server rendering then, gosh, I'm not sure :S you could share your configs & one example that times out |
I am using server rendering and MRI. We actually have an interesting set up where we are calling For a sanity check I configured it with: config.react.server_renderer_pool_size = 10
config.react.jsx_transformer_class = React::JSX::JSXTransformer To try and mimic the previous version and the same thing happens. I disabled pre-rendering and it loads fine, so it's definitely related to that. We were sometimes experiencing timeouts before the upgrade to 1.1. Since this is tightly integrated to a private application I'm not sure if I can pull out a failing test. But here's a stack trace of one of them that timed out(1.0) https://app.honeybadger.io/fault/44154/42efc9f4960c0e6686bd67844466a881 That prerender_component method looks like: def prerender_component(component_name, props)
react_component(component_name, deep_camelize_keys(props), { prerender: false })
end Any thoughts? |
I saw you have
Another idea for a work-around -- are you using NodeJS for ExecJS? If so, you could try |
Unfortunately I'm unable to look into the mini profiler results since it times out prior to the page rendering. Happy news though, adding I'll close for now since that seemed to resolve it. I'll reopen if it comes up again. Thanks for the quick responses and help @rmosolgo ! |
Sure thing glad that fixed it for now! 😬 |
My application Rails 4.2.2 (Ruby 2.2.2) on Heroku is timing out after upgrading. I'm using
Rack::Timeout
to kill requests after 10s and it hits it every time.Here's what I'm seeing in the logs:
When I rollback to 1.0 it goes away and I no longer timeout. I tried setting the transformer to the
JSXTransformer
to eliminateBabel
as a possibility and I still see the same issue so it looks like it's something else.Any idea what could be causing this?
The text was updated successfully, but these errors were encountered: