Async support #144

Closed
wants to merge 5 commits into
from

Conversation

Projects
None yet
3 participants
@bytenik
Contributor

bytenik commented Sep 17, 2012

No description provided.

@haf

This comment has been minimized.

Show comment Hide comment
@haf

haf Sep 17, 2012

Contributor

There goes my weekend

Contributor

haf commented Sep 17, 2012

There goes my weekend

@bytenik

This comment has been minimized.

Show comment Hide comment
@bytenik

bytenik Sep 17, 2012

Contributor

LOL. In all seriousness though, this is a very very minimalist approach -- under the hood, there's nothing async about it since the threads still block. This is merely so that we can start writing async handlers now and then have MT support it with an async pipeline later. (Instead of writing synchronous handlers now and then rewriting stuff when MT eventually has an async pipeline.)

Contributor

bytenik commented Sep 17, 2012

LOL. In all seriousness though, this is a very very minimalist approach -- under the hood, there's nothing async about it since the threads still block. This is merely so that we can start writing async handlers now and then have MT support it with an async pipeline later. (Instead of writing synchronous handlers now and then rewriting stuff when MT eventually has an async pipeline.)

@@ -8,7 +8,6 @@
[assembly: AssemblyVersion("2.6.3")]
[assembly: AssemblyFileVersion("2.6.3")]
-[assembly: AssemblyInformationalVersion("2.6.3.0647b3")]
+[assembly: AssemblyInformationalVersion("2.6.3.ffbce4")]

This comment has been minimized.

Show comment Hide comment
@haf

haf Sep 17, 2012

Contributor

This file has got to have badass code-churn ;).

@haf

haf Sep 17, 2012

Contributor

This file has got to have badass code-churn ;).

+ using (context.CreateScope())
+ {
+ var task = instance.Consume(context.Message);
+ task.Wait();

This comment has been minimized.

Show comment Hide comment
@haf

haf Sep 17, 2012

Contributor

So yeah, this is what you mean?

@haf

haf Sep 17, 2012

Contributor

So yeah, this is what you mean?

@drusellers

This comment has been minimized.

Show comment Hide comment
@drusellers

drusellers Mar 3, 2013

Member

Closing this Pull Request as the conversation has died. The Async pipeline is def interesting. Thanks for the conversation. :)

Member

drusellers commented Mar 3, 2013

Closing this Pull Request as the conversation has died. The Async pipeline is def interesting. Thanks for the conversation. :)

@drusellers drusellers closed this Mar 3, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment