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

Avoid loading extensions from classpath #3207

Closed
wants to merge 1 commit into from

Conversation

rafaelbey
Copy link
Contributor

@rafaelbey rafaelbey commented Oct 25, 2024

Avoid loading extensions over and over when parsing relational connections, and use the already loaded parser extensions.

@rafaelbey rafaelbey requested a review from a team as a code owner October 25, 2024 19:20
Copy link

Test Results

   475 files   -    579     475 suites   - 579   39m 19s ⏱️ - 1h 0m 30s
6 287 tests  - 6 523  6 221 ✔️  - 6 480  64 💤  - 45  0 ±0  2 🔥 +2 
9 479 runs   - 9 126  9 413 ✔️  - 9 083  64 💤  - 45  0 ±0  2 🔥 +2 

For more details on these errors, see this check.

Results for commit 07321c4. ± Comparison against base commit e7e709b.

@finos-admin
Copy link
Member

This PR is stale because it has been open for 30 days with no activity. Please remove stale label or add any comment to keep this open. Otherwise this will be closed in 5 days.

@finos-admin
Copy link
Member

This PR was closed because it has been inactive for 35 days. Please re-open if this PR is still relevant.

@rafaelbey rafaelbey deleted the use_extensions branch December 18, 2024 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants