helios-io / helios

reactive socket middleware for .NET

http://helios-io.github.io/

Apache License 2.0

453 stars 127 forks source link

got a System.OutOfMemoryException #59

Closed pinopino closed 8 years ago

pinopino commented 9 years ago

i'm interest in helios. but i got a System.OutOfMemoryException when i ran TimeService example on single local machine. sorry i can't support the stacktrace info, but below is the method throw this exception (it's in helios-1.4.1\src\Helios\Concurrency\Impl\DedicatedThreadPoolFiber.cs):

public void Add(Action op)
{
     if (Running)
         _threadPool.QueueUserWorkItem(() => Executor.Execute(op));
}

os: window 7 ultimate 64 bit memory: 16g

kklldog commented 8 years ago

me too.....

samcov commented 8 years ago

I too have seen this! I first saw it in Akka's TimeService sample, and wondered if it was Actor related.

I later tried Helios's version and the same problem occurred.

This is obviously a Helios issue, and it's a real show stopper!

samcov commented 8 years ago

Just noticed that Helios is depreciated in favor of DotNetty from Azure.

Aaronontheweb commented 8 years ago

75 related - looking into it.

Aaronontheweb commented 8 years ago

99% sure the issue here is the unnecessary allocation of new delegates.... This might be a 1-line fix. Let's see.

Aaronontheweb commented 8 years ago

Fixed as part of 2.1 using the new APIs.