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.

Sign up for GitHub

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

Jump to bottom

Rework discover: remove complex waiting logic due to clients have retry #1124

Closed
denis-tingaikin opened this issue Oct 27, 2021 · 2 comments
Closed

Rework discover: remove complex waiting logic due to clients have retry #1124

denis-tingaikin opened this issue Oct 27, 2021 · 2 comments
Assignees
@denis-tingaikin
Labels
ASAP The issue that blocking SOW items or core use-cases blocked Something is blocking this enhancement New feature or request

Comments

@denis-tingaikin
Copy link
Member

We should remove complex logic with timing in discover because with #1123 we don't need to wait for something.

Note: forwarder should not allocate resources on request. Resources should be allocated backward in these terms this change has sense.
@denis-tingaikin denis-tingaikin self-assigned this Oct 27, 2021
@denis-tingaikin denis-tingaikin added ASAP The issue that blocking SOW items or core use-cases enhancement New feature or request labels Oct 27, 2021
@denis-tingaikin
Copy link
Member Author

Internally discussed with @edwarnicke we should add this before p2mp to make things simple.

@denis-tingaikin
Copy link
Member Author

Blocked by #1123

@denis-tingaikin denis-tingaikin added the blocked Something is blocking this label Oct 27, 2021
@denis-tingaikin denis-tingaikin mentioned this issue Oct 31, 2021
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@denis-tingaikin denis-tingaikin

Labels
ASAP The issue that blocking SOW items or core use-cases blocked Something is blocking this enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@glazychev-art @denis-tingaikin