Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

Ut changes group size and indexsize tunable. #890

Closed
wants to merge 29 commits into from

Conversation

nihar1313
Copy link

No description provided.

maximalezhin and others added 29 commits May 28, 2021 04:35
Problem: CAS FOM locality index is calculated as a hash of
catalogue FID.

During performance profiling it has been found that the same
locality index is assigned to FOM if the CAS FOPs contain
the same FID. For instance, typical S3 client performs
operations on a single S3 bucket, which lead to limited set of
indecies (meta-index and ordinary index related to S3 bucket).
In that case, assigned localities' runq and waitq queues get
overloaded, while other localities' queues are not populated
by incoming CAS FOPs. It increases CAS FOP processing latency.

Solution: Locality index is implemented using incremental
static counter. It's updated by +1 value each time cas_fom_home_locality()
function is called.

Signed-off-by: Maxim Malezhin <maxim.malezhin@seagate.com>
Signed-off-by: Nihar <nihar.nayak@seagate.com>
@stale
Copy link

stale bot commented Jul 31, 2021

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @huanghua78 @atulsdeshmukh2312 @mehjoshi for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

@cla-bot
Copy link

cla-bot bot commented Aug 2, 2021

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Nihar Nayak.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email email@example.com
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@stale stale bot removed the needs-attention label Aug 2, 2021
@nihar1313
Copy link
Author

This is no longer valid, new PR has been raised

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants