Skip to content

[#8445] improvement(core): Share and cache class loader instance for catalog#8536

Closed
yuqi1129 wants to merge 7 commits intoapache:mainfrom
yuqi1129:issue_8445
Closed

[#8445] improvement(core): Share and cache class loader instance for catalog#8536
yuqi1129 wants to merge 7 commits intoapache:mainfrom
yuqi1129:issue_8445

Conversation

@yuqi1129
Copy link
Contributor

What changes were proposed in this pull request?

  1. Add a reference count to show how many catalogs a class loader is shared with.
  2. Do not immediately close a class loader when it is not used by any catalog.

Why are the changes needed?

For better performance.

Fix: #8445

Does this PR introduce any user-facing change?

N/A.

How was this patch tested?

Existing tests.

@yuqi1129 yuqi1129 marked this pull request as ready for review September 12, 2025 11:50
@yuqi1129 yuqi1129 self-assigned this Sep 12, 2025
@yuqi1129 yuqi1129 requested review from FANNG1 and mchades September 17, 2025 03:41
@yuqi1129
Copy link
Contributor Author

Close it temporarily as it's imperfect.

@yuqi1129 yuqi1129 closed this Oct 15, 2025
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 this pull request may close these issues.

[FEATURE] Cache and reuse catalog classloader

1 participant