-
Notifications
You must be signed in to change notification settings - Fork 28.5k
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
[SPARK-3710] Fix Yarn integration tests on Hadoop 2.2. #2682
Conversation
It seems some dependencies are not declared when pulling the 2.2 test dependencies, so we need to add them manually for the Yarn cluster to come up. These don't seem to be necessary for 2.3 and beyond, so restrict them to the hadoop-2.2 profile.
Test FAILed. |
Jenkins, retest this please. |
Is it the Hadoop 2.2 test code that needs these things, or Spark test code? I am guessing it is the former, and this is because test deps aren't transitive but happen to get covered by Hadoop 2.3 non-test transitive dependencies otherwise? |
QA tests have started for PR 2682 at commit
|
Yes, it's the hadoop side that's missing those. They were added in the 2.3 release, so this is just needed when running against 2.2. I haven't looked at details of the hadoop poms, though. |
QA tests have finished for PR 2682 at commit
|
Test PASSed. |
Would you mind adding a comment here with the JIRA number in there and explaining these are for YARN integration tests? |
QA tests have started for PR 2682 at commit
|
Tests timed out for PR 2682 at commit |
Test FAILed. |
Failed some Spark SQL test, flaky? |
Jenkins, retest this please. |
Test FAILed. |
And now, infrastructure issues? |
Jenkins, retest this please. |
QA tests have started for PR 2682 at commit
|
QA tests have finished for PR 2682 at commit
|
Test PASSed. |
Great, let's take it for a spin. |
It seems some dependencies are not declared when pulling the 2.2
test dependencies, so we need to add them manually for the Yarn
cluster to come up.
These don't seem to be necessary for 2.3 and beyond, so restrict
them to the hadoop-2.2 profile.