Abhijeetsng97 opened a new pull request, #13574:
URL: https://github.com/apache/gravitino/pull/13574
<!--
1. Title: [#<issue>] <type>(<scope>): <subject>
Examples:
- "[#123] feat(operator): Support xxx"
- "[#233] fix: Check null before access result in xxx"
- "[MINOR] refactor: Fix typo in variable name"
- "[MINOR] docs: Fix typo in README"
- "[#255] test: Fix flaky test NameOfTheTest"
Reference: https://www.conventionalcommits.org/en/v1.0.0/
2. If the PR is unfinished, please mark this PR as draft.
-->
### What changes were proposed in this pull request?
When entity caching is disabled (gravitino.cache.enabled=false), Gravitino
uses NoOpsCache, which
acquires a single shared lock for all entity reads. The lock remains held
during database access,
forcing unrelated reads to execute sequentially.
Under concurrent load, this slows down metadata reads and increases request
latency. A slow database
lookup blocks other requests, potentially exhausting request threads and
causing timeouts.
Since NoOpsCache stores no cached state, this locking is unnecessary. Reads
should execute
concurrently when caching is disabled.
### Why are the changes needed?
(Please clarify why the changes are needed. For instance,
1. If you propose a new API, clarify the use case for a new API.
2. If you fix a bug, describe the bug.)
Fix: #(issue)
#13573
### How was this patch tested?
locally and via unit tests
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
