Skip to content

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
@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