-
Notifications
You must be signed in to change notification settings - Fork 29k
fix different versions of commons-lang dependency and apache/spark#746 addendum #754
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
Conversation
|
Can one of the admins verify this patch? |
|
@srowen have time to review the code? |
|
Ah right I neglected to update the SBT build yesterday. This looks like just a mirror of the two recent Maven build changes? Then LGTM. @pwendell |
|
Yea I think this is just bringing it up to par. I'll merge it. Thanks. |
|
Jenkins, test this please. |
|
Merged build triggered. |
|
Merged build started. |
|
Merged build finished. All automated tests passed. |
|
All automated tests passed. |
Author: witgo <[email protected]> Closes #754 from witgo/commons-lang and squashes the following commits: 3ebab31 [witgo] merge master f3b8fa2 [witgo] merge master 2083fae [witgo] repeat definition 5599cdb [witgo] multiple version of sbt dependency c1b66a1 [witgo] fix different versions of commons-lang dependency (cherry picked from commit bae07e3) Signed-off-by: Patrick Wendell <[email protected]>
|
Just a heads up, this patch caused an issue which is currently blocking the release: |
…ndum Author: witgo <[email protected]> Closes apache#754 from witgo/commons-lang and squashes the following commits: 3ebab31 [witgo] merge master f3b8fa2 [witgo] merge master 2083fae [witgo] repeat definition 5599cdb [witgo] multiple version of sbt dependency c1b66a1 [witgo] fix different versions of commons-lang dependency
No description provided.