search

1602 / compound

MVC framework. Built on Node.JS. Works on server and browser.
http://compoundjs.com
1.6k stars 182 forks source link

Controller is not ready to handle another request #568

Closed aleksandrmelnyk closed 11 years ago

aleksandrmelnyk commented 11 years ago

Sometimes I am receiving error 'Controller is not ready to handle another request'. What could be reason of this?

aleksandrmelnyk commented 11 years ago

That controller is pretty big, by the way, but still I am not sure meaning of that exception.

1602 commented 11 years ago

Hey @mamayoleksandr could you please provide more details on this? Isolated case would be really helpful.

aleksandrmelnyk commented 11 years ago

HI Anatoliy,

So we have ajax gis application, where client is doing kinda 20-30 requsts when web page is openeded. And sometimes 2-3 requests returned 500 error (see screenshot) selection_033

We are using compounds.js of version 1.1.6-2 for already ~5 months, but I only started to see this error for last week sometimes. The controller is noevel. Server machine is not overloaded (50 % free RAM, enough space in hard disk etc.).

Hope this would clarify situation slightly.

Thank you,

Alex.

1602 commented 11 years ago

Thanks, I will try to reproduce. Actually we are getting this error sometimes on high loaded project accidentally, but too few times to worry too much about it.

aleksandrmelnyk commented 11 years ago

today I saw it quite often. Can it be somehow connected to controller pool? may be there is some workaround..?

1602 commented 11 years ago

@mamayoleksandr okay, I guess I found it, controller was not release correctly when there was an error, just need to build the test case to prove it.

1602 commented 11 years ago

@mamayoleksandr should be fixed now (pull kontroller module).

aleksandrmelnyk commented 11 years ago

Hi Anatoliy, sorry, do you mean this one https://github.com/1602/kontroller, or update through npm?

Thank you,

Alex.

2013/9/7 Anatoliy Chakkaev notifications@github.com

@mamayoleksandr https://github.com/mamayoleksandr should be fixed now (pull kontroller module).

— Reply to this email directly or view it on GitHubhttps://github.com/1602/compound/issues/568#issuecomment-23973405 .

anatoliychakkaev commented 11 years ago

npm

On Sat, Sep 7, 2013 at 5:07 AM, Oleksandr Melnyk notifications@github.com wrote:

Hi Anatoliy, sorry, do you mean this one https://github.com/1602/kontroller, or update through npm? Thank you, Alex. 2013/9/7 Anatoliy Chakkaev notifications@github.com

@mamayoleksandr https://github.com/mamayoleksandr should be fixed now (pull kontroller module).

— Reply to this email directly or view it on GitHubhttps://github.com/1602/compound/issues/568#issuecomment-23973405 .

Reply to this email directly or view it on GitHub: https://github.com/1602/compound/issues/568#issuecomment-23981194

aleksandrmelnyk commented 11 years ago

Hi Anataliy,

I have updated through npm, but cannot see changes in sources code of kontroller..

anatoliychakkaev commented 11 years ago

Try to use compound from git/master: npm install 1602/compound

On Mon, Sep 9, 2013 at 9:34 AM, Oleksandr Melnyk notifications@github.comwrote:

Hi Anataliy,

I have updated through npm, but cannot see changes in sources code of kontroller..

— Reply to this email directly or view it on GitHubhttps://github.com/1602/compound/issues/568#issuecomment-24053567 .

Thanks, Anatoliy Chakkaev

aleksandrmelnyk commented 11 years ago

Updated, so far didn't saw the issue.

Thank you.

icaliman commented 9 years ago

This issue happens to me with compound@1.1.15

randunel commented 9 years ago

This issue was fixed for me (v1.1.15), can you come up with a way to reproduce it?

icaliman commented 9 years ago

It happens on high loaded server, in some routes I do requests to other services and it takes ~1sec to send response

alanpca commented 8 years ago

@anatoliychakkaev we are seeing this back in 1.1.18.