alluxio
alluxio copied to clipboard
Implement SharedResourcePool
What changes are proposed in this pull request?
Implemented SharedResourcePool
, which unlike DynamicResourcePool
, shares resources across multiple acquire calls and never blocks.
Why are the changes needed?
We might want to replace the current ClientPool
model to share clients within the same FileSystemContext
, which might help with issues like https://github.com/Alluxio/alluxio/issues/15893
Also, sharing client makes sense as clients are pretty much stateless and already synchronized. There's very little benefit creating many instances of the client anyway, since the underlying connection channels are already shared.
Does this PR introduce any user facing changes?
No
@dbw9580 Would you mind taking a look at this? Appreciate your feedback.
@ssz1997 For your visibility here.
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.