Race in LookPool may cause used Resources to be evicted

Alluxio / alluxio

Alluxio, data orchestration for analytics and machine learning in the cloud

https://www.alluxio.io

Apache License 2.0

6.87k stars 2.94k forks source link

Race in LookPool may cause used Resources to be evicted #18678

Open ZanderXu opened 2 months ago

ZanderXu commented 2 months ago

This PR is mainly used to fix a race in LookPool.class, which may cause some used Resources to be evicted by the Evictor.

Before evicting Resources from LookPool, the Evictor should check if the to be deleted Resource changed. And This remove operation should be synced with the getResource.

alluxio-bot commented 2 months ago

Thank you for your pull request. In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement (CLA). It's all electronic and will take just a few minutes. Please download CLA form here, sign, and e-mail back to cla@alluxio.org

alluxio-bot commented 2 months ago

Automated checks report:

PR title follows the conventions: FAIL
- The title of the PR does not pass all the checks. Please fix the following issues:
  - First word of title ("A") is not an imperative verb. Please use one of the valid words
  - Supported title prefixes are: [WIP], [SMALLFIX], [DOCFIX]
Commits associated with Github account: PASS

Some checks failed. Please fix the reported issues and reply alluxio-bot, check this please to re-run checks.