alluxio icon indicating copy to clipboard operation
alluxio copied to clipboard

Implement SharedResourcePool

Open YangchenYe323 opened this issue 2 years ago • 2 comments

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

YangchenYe323 avatar Jul 26 '22 23:07 YangchenYe323

@dbw9580 Would you mind taking a look at this? Appreciate your feedback.

YangchenYe323 avatar Jul 27 '22 00:07 YangchenYe323

@ssz1997 For your visibility here.

YangchenYe323 avatar Aug 05 '22 19:08 YangchenYe323

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.

github-actions[bot] avatar Feb 01 '23 15:02 github-actions[bot]

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.

github-actions[bot] avatar Jun 16 '23 15:06 github-actions[bot]