Skip to content
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

[SPIKE] Async API from Highlevel #151

Closed
wants to merge 1 commit into from

Conversation

danielmarbach
Copy link
Collaborator

DO NOT MERGE

Opposed to #149 this PR aims to explore an async API from a high level perspective first. Instead of changing low level infrastructure like NetworBinaryWriter it starts from IModel.

The idea of this spike is to show that we can stay as true as possible to the current API design and "just" make it async enabled without requiring a whole lot of new API design changes

@michaelklishin thoughts?

@michaelklishin
Copy link
Member

Thank you! We're quite busy this week, will get to this PR eventually.

@danielmarbach
Copy link
Collaborator Author

Don't worry. Enjoying my life on thailand. Take your time

Am 13.01.2016 um 17:54 schrieb Michael Klishin [email protected]:

Thank you! We're quite busy this week, will get to this PR eventually.


Reply to this email directly or view it on GitHub.

@danielmarbach danielmarbach mentioned this pull request Jan 24, 2016
@danielmarbach
Copy link
Collaborator Author

I'm back from Thailand and hopefully more responsive

Am 13.01.2016 um 11:54 schrieb Michael Klishin [email protected]:

Thank you! We're quite busy this week, will get to this PR eventually.


Reply to this email directly or view it on GitHub.

@michaelklishin
Copy link
Member

Thanks. Unfortunately we are still quite jammed.

@danielmarbach
Copy link
Collaborator Author

@michaelklishin ping me when you want me to help you guys out move the async/await initiative forward

@michaelklishin
Copy link
Member

@danielmarbach we will, thank you. I'm going to post a plan for the .NET client on Monday or so next week.

@michaelklishin michaelklishin self-assigned this Feb 23, 2016
@michaelklishin
Copy link
Member

I've recently reviewed this with someone and will close this and post some thoughts about our plan to #83. Thank you very much for providing this spike PR, it's influenced our thinking on the subject.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants