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

async client #13

Closed
oroulet opened this issue Aug 8, 2014 · 3 comments
Closed

async client #13

oroulet opened this issue Aug 8, 2014 · 3 comments
Assignees

Comments

@oroulet
Copy link
Member

oroulet commented Aug 8, 2014

Client must ne ported to asio
-- to make subscription client work,
-- be portable to windows
-- to not hang if responses do not arrive in expected order

@arykovanov
Copy link
Member

Refactored client code. Removed most of layers in client library: Now there will be one central component - BinaryServer. It will have a map of request handles and callback for processing responses in the same manner as in server OpcTcpMessages.
Algorithm will be the next:

  1. Form Request.
  2. Put into map <requestID, callback>.
  3. Request is sent to server.
  4. When response is came then corresponding requestID pops from map.
  5. Deserialization and call callback.

@oroulet
Copy link
Member Author

oroulet commented Aug 18, 2014

Sounds correct for me.

On 18 August 2014 09:47, Alexander Rykovanov notifications@github.com
wrote:

Refactored client code. Removed most of layers in client library: Now
there will be one central component - BinaryServer. It will have a set of
request handles and callback for processing responses in the same manner as
in server OpcTcpMessages.
Algorithm will be the next:

  1. Form Request.
  2. Put into map .
  3. Request is sent to server.
  4. When response is came then corresponding requestID pops from map.
  5. Deserialization and call callback.


Reply to this email directly or view it on GitHub
#13 (comment).

@oroulet
Copy link
Member Author

oroulet commented Sep 3, 2014

implemented

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

No branches or pull requests

2 participants