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

[SPARK-7558] Guard against direct uses of FunSuite / FunSuiteLike #6510

Closed
wants to merge 3 commits into from

Conversation

andrewor14
Copy link
Contributor

This is a follow-up patch to #6441.

@andrewor14 andrewor14 force-pushed the extends-funsuite-check branch from 6f6d546 to 48874dd Compare May 29, 2015 22:54
@andrewor14
Copy link
Contributor Author

@rxin I couldn't figure out how to make the regex start with a lower case character / include spaces, e.g. extends FunSuite

@SparkQA
Copy link

SparkQA commented May 30, 2015

Test build #33780 has finished for PR 6510 at commit 48874dd.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented May 30, 2015

Test build #33779 has finished for PR 6510 at commit 6f6d546.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@andrewor14
Copy link
Contributor Author

Do not merge this yet because it will fail tests in flume-sink after the latest commits in master.

Andrew Or added 2 commits May 29, 2015 18:50
See the in-code comment for an explanation of why this is necessary.
@SparkQA
Copy link

SparkQA commented May 30, 2015

Test build #33790 has finished for PR 6510 at commit 6618b46.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@rxin
Copy link
Contributor

rxin commented May 30, 2015

LGTM.

@rxin
Copy link
Contributor

rxin commented May 30, 2015

Merged in master.

@asfgit asfgit closed this in 609c492 May 30, 2015
@andrewor14 andrewor14 deleted the extends-funsuite-check branch May 30, 2015 06:06
andrewor14 pushed a commit to andrewor14/spark that referenced this pull request Jun 2, 2015
This is a follow-up patch to apache#6441.

Author: Andrew Or <[email protected]>

Closes apache#6510 from andrewor14/extends-funsuite-check and squashes the following commits:

6618b46 [Andrew Or] Exempt SparkSinkSuite from the FunSuite check
99d02ac [Andrew Or] Merge branch 'master' of github.com:apache/spark into extends-funsuite-check
48874dd [Andrew Or] Guard against direct uses of FunSuite / FunSuiteLike
andrewor14 pushed a commit to andrewor14/spark that referenced this pull request Jun 3, 2015
This is a follow-up patch to apache#6441.

Author: Andrew Or <[email protected]>

Closes apache#6510 from andrewor14/extends-funsuite-check and squashes the following commits:

6618b46 [Andrew Or] Exempt SparkSinkSuite from the FunSuite check
99d02ac [Andrew Or] Merge branch 'master' of github.com:apache/spark into extends-funsuite-check
48874dd [Andrew Or] Guard against direct uses of FunSuite / FunSuiteLike

Conflicts:
	scalastyle-config.xml
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
This is a follow-up patch to apache#6441.

Author: Andrew Or <[email protected]>

Closes apache#6510 from andrewor14/extends-funsuite-check and squashes the following commits:

6618b46 [Andrew Or] Exempt SparkSinkSuite from the FunSuite check
99d02ac [Andrew Or] Merge branch 'master' of github.com:apache/spark into extends-funsuite-check
48874dd [Andrew Or] Guard against direct uses of FunSuite / FunSuiteLike
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
This is a follow-up patch to apache#6441.

Author: Andrew Or <[email protected]>

Closes apache#6510 from andrewor14/extends-funsuite-check and squashes the following commits:

6618b46 [Andrew Or] Exempt SparkSinkSuite from the FunSuite check
99d02ac [Andrew Or] Merge branch 'master' of github.com:apache/spark into extends-funsuite-check
48874dd [Andrew Or] Guard against direct uses of FunSuite / FunSuiteLike
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.

3 participants