Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce CacheImpl lock contention #5973

Closed
shawkins opened this issue May 1, 2024 · 0 comments · Fixed by #5974
Closed

Reduce CacheImpl lock contention #5973

shawkins opened this issue May 1, 2024 · 0 comments · Fixed by #5974
Assignees
Milestone

Comments

@shawkins
Copy link
Contributor

shawkins commented May 1, 2024

See #5699

When io.fabric8.kubernetes.client.informers.cache.Lister#list is used with a namespace under heavy concurrent load, the CacheImpl#byIndex becomes a performance bottleneck due to synchronized method contention.

shawkins added a commit to shawkins/kubernetes-client that referenced this issue May 1, 2024
closes: fabric8io#5973

Signed-off-by: Steve Hawkins <shawkins@redhat.com>
shawkins added a commit to shawkins/kubernetes-client that referenced this issue May 1, 2024
closes: fabric8io#5973

Signed-off-by: Steve Hawkins <shawkins@redhat.com>
shawkins added a commit to shawkins/kubernetes-client that referenced this issue May 7, 2024
closes: fabric8io#5973

Signed-off-by: Steve Hawkins <shawkins@redhat.com>
shawkins added a commit to shawkins/kubernetes-client that referenced this issue May 8, 2024
closes: fabric8io#5973

Signed-off-by: Steve Hawkins <shawkins@redhat.com>
shawkins added a commit to shawkins/kubernetes-client that referenced this issue May 8, 2024
closes: fabric8io#5973

Signed-off-by: Steve Hawkins <shawkins@redhat.com>
@manusa manusa mentioned this issue May 9, 2024
11 tasks
manusa pushed a commit to shawkins/kubernetes-client that referenced this issue May 9, 2024
closes: fabric8io#5973

Signed-off-by: Steve Hawkins <shawkins@redhat.com>
@manusa manusa added this to the 6.13.0 milestone May 9, 2024 — with automated-tasks
manusa pushed a commit that referenced this issue May 9, 2024
Reduce CacheImpl lock contention
---
Add BasicItemStoreTest
---
Expand CacheTest
---
Back to the future - fix copyright year
---
fix: reduce the locking of methods reading indexes

closes: #5973

Signed-off-by: Steve Hawkins <shawkins@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants