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

Move JDBC drivers to a shared location #3257

Merged
merged 3 commits into from
Sep 27, 2023
Merged

Move JDBC drivers to a shared location #3257

merged 3 commits into from
Sep 27, 2023

Conversation

hansva
Copy link
Contributor

@hansva hansva commented Sep 26, 2023

In this PR all JDBC drivers that we include in our distribution have been moved to lib/jdbc.

The Variable that can be used to add external JDBC drivers has been changed to HOP_SHARED_JDBC_FOLDERS and accepts a comma separated list of folders.

The relevant documentation changes have happend

Added the option for transforms/actions to also load the JDBC driver folder if needed (mainly for bulkloaders)
Grouped the classloaders of the bulkloaders and their relevant Database types to avoid having the JDBC drivers in separate classloaders.


Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
  • If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
  • Mention the appropriate issue in your description (for example: addresses #123), if applicable.

To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

@hansva hansva merged commit bffdff2 into apache:master Sep 27, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant