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

Improve how lease API is used #877

Closed
OlegDokuka opened this issue Jun 26, 2020 · 0 comments · Fixed by #885
Closed

Improve how lease API is used #877

OlegDokuka opened this issue Jun 26, 2020 · 0 comments · Fixed by #885
Assignees
Labels
enhancement superseded Issue is superseded by another

Comments

@OlegDokuka
Copy link
Member

OlegDokuka commented Jun 26, 2020

The mechanics of using leasing from a requester's perspective are somewhat convoluted. You have to manually deal with waiting for leases to be acquired initially, and then separately track new leases, and finally connecting that into further requests. In addition a server side requester cannot even do that because it has no way to obtain a leases receiver for the current connection.

The idea is to try and encapsulate the acquiring of a lease in an RSocketProxy or perhaps build it into RSocketRequester such that an application can make requests as usual but internally that would involve acquiring a lease. This would make it straight forward to make requests with leases from both sides.

@OlegDokuka OlegDokuka added this to the 1.1 M1 milestone Jun 26, 2020
@OlegDokuka OlegDokuka linked a pull request Jul 7, 2020 that will close this issue
@rstoyanchev rstoyanchev modified the milestones: 1.1 M1, 1.1 M2 Aug 10, 2020
@OlegDokuka OlegDokuka modified the milestones: 1.1 M2, 1.x Backlog Sep 9, 2020
@OlegDokuka OlegDokuka modified the milestones: 1.x Backlog, 1.1.1 Nov 3, 2020
@OlegDokuka OlegDokuka self-assigned this Feb 17, 2021
@rstoyanchev rstoyanchev added the superseded Issue is superseded by another label Jun 9, 2021
@rstoyanchev rstoyanchev removed this from the 1.1.1 milestone Jun 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement superseded Issue is superseded by another
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants