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

Mark test cases that require spark2 #951

Merged
merged 2 commits into from
Aug 28, 2018
Merged

Conversation

anwar6953
Copy link
Contributor

@anwar6953 anwar6953 commented Aug 28, 2018

Upgrade to JUnit 4.12 to have support for multiple CategoryExcludes.
Create a RequiresSpark and RequiresSpark2 to represent categories that can be excluded if a particular distro does not support Spark or Spark2.
Also added an AmbariSuite for running on Ambari that is running with Spark1.

Note: This depends on https://github.com/caskdata/cdap-integration-tests/pull/950

…ories. Use RequiresSpark where we were previously excluding tests that used Spark.
@anwar6953 anwar6953 force-pushed the CDAP-14132-spark-ignores branch from ea338b5 to db89b37 Compare August 28, 2018 18:04
…resSpark2 category from test suites for distros that do not support Spark2.
@anwar6953 anwar6953 force-pushed the CDAP-14132-spark-ignores branch from db89b37 to 6bbff2e Compare August 28, 2018 18:10
Copy link
Contributor

@chtyim chtyim left a comment

Choose a reason for hiding this comment

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

LGTM

@anwar6953 anwar6953 merged commit 2952397 into develop Aug 28, 2018
@anwar6953 anwar6953 deleted the CDAP-14132-spark-ignores branch August 28, 2018 18:28
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