Uh oh!
There was an error while loading. Please reload this page.
[SPARK-5979][SPARK-6032] Smaller safer --packages fix - #4802
Closed
brkyvz wants to merge 2 commits into
Closed
Conversation
tdas
commented
Feb 27, 2015
Contributor
I tested your PR branch and its works pretty well! |
SparkQA
commented
Feb 27, 2015
Test build #28036 has finished for PR 4802 at commit
|
tdas
commented
Feb 27, 2015
Contributor
LGTM |
pwendell
commented
Feb 27, 2015
Contributor
LGTM |
JoshRosen
commented
Feb 27, 2015
Contributor
Mind maybe copying some of the description from the other PR into this one so that the commit message is useful? |
pwendell
commented
Feb 28, 2015
Contributor
Pulling this in - thanks Burak! |
asfgit pushed a commit
that referenced
this pull request
Feb 28, 2015
pwendell tdas This is the safer parts of PR #4754: - SPARK-5979: All dependencies with the groupId `org.apache.spark` passed through `--packages`, were being excluded from the dependency tree on the assumption that they would be in the assembly jar. This is not the case, therefore the exclusion rules had to be defined more explicitly. - SPARK-6032: Ivy prints a whole lot of logs while retrieving dependencies. These were printed to `System.out`. Moved the logging to `System.err`. Author: Burak Yavuz <brkyvz@gmail.com> Closes#4802 from brkyvz/simple-streaming-fix and squashes the following commits: e0f38cb [Burak Yavuz] Merge branch 'master' of github.com:apache/spark into simple-streaming-fix bad921c [Burak Yavuz] [SPARK-5979][SPARK-6032] Smaller safer fix (cherry picked from commit 6d8e5fb) Signed-off-by: Patrick Wendell <patrick@databricks.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@pwendell@tdas
This is the safer parts of PR #4754:
org.apache.sparkpassed through--packages, were being excluded from the dependency tree on the assumption that they would be in the assembly jar. This is not the case, therefore the exclusion rules had to be defined more explicitly.System.out. Moved the logging toSystem.err.