duongkame opened a new pull request, #5427: URL: https://github.com/apache/ozone/pull/5427
## What changes were proposed in this pull request? OM's getKeyInfo API internally does the following steps: 1. Resolve bucket link: confirms that bucket/volume exists, resolve the bucket's real name if the bucket param points to a soft link. 2. ACL check: check if the user has access to the requested resource. 3. Read key metadata from rocks db. 4. Get block container location cache/SCM. 5. Generate block token for read. Although during (1), bucket/volume info is already retrieved from db/cache, step (2) and (3) still re-read the same information to extract different sub-attribute. e.g. (2) needs to read bucket/volume to get bucket/volume owner name, (3) needs to get bucket layout. Those redundant reads unnecessarily add a significant cost to API, given that each read involves locking, encoding/decoding results... ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-9297 ## How was this patch tested? I did a quick load test in an internal cluster. This change helped the peak OPPS of OM pure reads (getKeyInfo) increase from **150K** to **190K**. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
