Skip to content

[perf][COM][kerberos] optimize keytab file caching to prevent full gc issues#972

Merged
casionone merged 3 commits intodev-2.0.0from
dev-2.0.0-ketab-cache
Apr 7, 2026
Merged

[perf][COM][kerberos] optimize keytab file caching to prevent full gc issues#972
casionone merged 3 commits intodev-2.0.0from
dev-2.0.0-ketab-cache

Conversation

@v-kkhuang
Copy link
Copy Markdown

What is the purpose of the change

Background/Problem:
The current keytab file caching mechanism causes Full GC issues due to improper memory management. This affects system performance and stability, especially in Kerberos authentication scenarios with high concurrency.

Purpose of Change:
To address this performance issue, this PR optimizes the keytab file caching mechanism by adding a more efficient keytabFile cache implementation to reduce memory pressure and prevent Full GC.

Value/Impact:
After the change, system memory usage is optimized, Full GC frequency is significantly reduced, and overall system stability is improved in Kerberos authentication environments.

Related issues/PRs

Related issues: close #971
Related pr:none

Brief change log

  • Optimize keytab file caching mechanism to prevent Full GC
  • Add efficient keytabFile cache implementation
  • Improve memory management for Kerberos authentication

Checklist

  • I have read the Contributing Guidelines on pull requests.
  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible
  • If this is a code change: I have written unit tests to fully verify the new behavior.

@v-kkhuang v-kkhuang added the enhancement New feature or request label Apr 7, 2026
Copy link
Copy Markdown

@casionone casionone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@casionone casionone merged commit 00a099c into dev-2.0.0 Apr 7, 2026
18 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants