-
Notifications
You must be signed in to change notification settings - Fork 119
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
Update jar directory locations and versions in config code #86
Conversation
@tknandu, It will cover your contributions to all Microsoft-managed open source projects. |
@tknandu, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you re-run the Spark Connector (Scala) notebook it's in entirety and change the JARs configuration to include all four JARs? Could you also include updating the PySpark version? Thanks!
Have addressed your comments @dennyglee |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - thanks!
Updating the configurations to use the most recent version of the jars and also ensure the directory location of the jars matches that specified in the wiki