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

MINOR: possibility to include mysql-connector jar in final tar.gz #668

Merged
merged 2 commits into from
Feb 17, 2020

Conversation

akatona84
Copy link
Contributor

This functionality is tied to with-mysql-jar profile

This functionality is tied to `with-mysql-jar` profile
@raju-saravanan
Copy link
Collaborator

@akatona84: May I know the motivation behind this ?

@akatona84
Copy link
Contributor Author

@akatona84: May I know the motivation behind this ?

Yes, it is required for cdp runtime. Libs dir won't be writeable by schemaregistry (otherwise bootstrap would download this jar)

@raju-saravanan
Copy link
Collaborator

@akatona84: Alright. How will the oracle ojdbc dependencies be dealt with in CDP runtime?

@akatona84
Copy link
Contributor Author

@akatona84: Alright. How will the oracle ojdbc dependencies be dealt with in CDP runtime?

I think it wasn't provided in csp either. User will copy them in place.

Change-Id: Id94973ec431846481a7cdee75e958ff0ac83d707
Copy link
Collaborator

@raju-saravanan raju-saravanan left a comment

Choose a reason for hiding this comment

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

+1 LGTM, thanks @akatona84 for raising this PR

@akatona84 akatona84 merged commit 9e33b2b into hortonworks:master Feb 17, 2020
@akatona84 akatona84 deleted the mysql-jar branch February 17, 2020 19:22
raju-saravanan referenced this pull request in raju-saravanan/registry Apr 21, 2020
raju-saravanan referenced this pull request in raju-saravanan/registry Apr 21, 2020
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.

2 participants