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 request processing in Northbound #405

Closed
sergii-iakovenko opened this issue Mar 27, 2018 · 1 comment
Assignees

Comments

@sergii-iakovenko
Copy link
Collaborator

@sergii-iakovenko sergii-iakovenko commented Mar 27, 2018

Re-implement the Northbound and Kafka communication approach.

The potential solution looks like:

  1. Create a new layer between NB services and kafka producers/consumers. It will manage all async requests and should return CompletableFuture.
  2. Create simple mechanism to get the response.
  3. The response dispatcher may reject new request if there're too many requests being processed. (to be discussed)
  4. Use the CompletableFuture as the REST method result.
  • The response dispatcher should subscribe on DeferredResult events
    (onTimeout and others) and react correspondingly. E.g. removes a correlationId->CompletableFuture entry if it waits a response for too long (timeout happens).
@nikitacherevko

This comment has been minimized.

Copy link
Collaborator

@nikitacherevko nikitacherevko commented Jun 11, 2018

Updated description due to latest changes.

@carmine carmine added the area/arch label Jun 27, 2018
nikitacherevko added a commit that referenced this issue Jul 5, 2018
nikitacherevko added a commit that referenced this issue Jul 5, 2018
nikitacherevko added a commit that referenced this issue Jul 5, 2018
nikitacherevko added a commit that referenced this issue Jul 5, 2018
nikitacherevko added a commit that referenced this issue Jul 6, 2018
nikitacherevko added a commit that referenced this issue Jul 9, 2018
nikitacherevko added a commit that referenced this issue Jul 9, 2018
nikitacherevko added a commit that referenced this issue Sep 19, 2018
nikitacherevko added a commit that referenced this issue Sep 19, 2018
nikitacherevko added a commit that referenced this issue Sep 19, 2018
nikitacherevko added a commit that referenced this issue Sep 19, 2018
nikitacherevko added a commit that referenced this issue Sep 19, 2018
nikitacherevko added a commit that referenced this issue Sep 21, 2018
nikitacherevko added a commit that referenced this issue Sep 21, 2018
nikitacherevko added a commit that referenced this issue Sep 21, 2018
nikitacherevko added a commit that referenced this issue Sep 21, 2018
nikitacherevko added a commit that referenced this issue Sep 24, 2018
nikitacherevko added a commit that referenced this issue Sep 24, 2018
nikitacherevko added a commit that referenced this issue Sep 24, 2018
nikitacherevko added a commit that referenced this issue Sep 24, 2018
nikitacherevko added a commit that referenced this issue Sep 24, 2018
nikitacherevko added a commit that referenced this issue Sep 24, 2018
nikitacherevko added a commit that referenced this issue Sep 25, 2018
nikitacherevko added a commit that referenced this issue Sep 25, 2018
nikitacherevko added a commit that referenced this issue Sep 28, 2018
nikitacherevko added a commit that referenced this issue Sep 28, 2018
@nikitacherevko nikitacherevko moved this from In progress to On Review in Open-Kilda Board Sep 28, 2018
nikitacherevko added a commit that referenced this issue Sep 28, 2018
nikitacherevko added a commit that referenced this issue Oct 1, 2018
nikitacherevko added a commit that referenced this issue Oct 4, 2018
nikitacherevko added a commit that referenced this issue Oct 4, 2018
nikitacherevko added a commit that referenced this issue Oct 4, 2018
nikitacherevko added a commit that referenced this issue Oct 16, 2018
#405: Refactored NB services to work with async requests
nikitacherevko added a commit that referenced this issue Oct 16, 2018
nikitacherevko added a commit that referenced this issue Oct 16, 2018
#405: Implemented async request processing for NB
@nikitacherevko nikitacherevko moved this from On Review to Done in Sep in Open-Kilda Board Nov 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.